Re: [Tuxpaint-devel] New "PACKAGE_ONLY" Makefile variable

[Will T. <wj...@en...>]

Looks great. The Flatpak package has a workaround where it sets
ARCH_INSTALL to 'install-man install-importscript' and then manually
installs the icons etc. I haven't tested but it looks like I could remove
this and replace it with PACKAGE_ONLY=yes.

I would suggest that the PACKAGE_ONLY behaviour should be the default –
I've never seen any other software using the xdg-icon-resource or
xdg-desktop-menu tools at build/install time. I don't know how common it is
to manually install Tux Paint to a location that is not in the default XDG
search path – i.e. with a prefix other than /usr, /usr/local, or
$HOME/.local – all of which will work in the new PACKAGE_ONLY=yes path.

On Wed, 7 Jun 2023 at 08:46, Bill Kendrick <nb...@so...> wrote:

>
> For a while now, Tux Paint has been using XDG tools (like
> `xdg-icon-resource` and `xdg-desktop-menu`) to install & uninstall
> launcher icons in the user's desktop GUI.
>
> However, they apparently aren't flexible enough to play nicely with
> folks who are trying to put together a package -- specifically, it
> was Tim who helps maintain the Slackware Linux package, who pointed
> out some workarounds that have been necessary to prevent Tux Paint
> from 'polluting' their local environment.
>
> I received a Makefile patch today, which I applied to the master
> branch (what will become 0.9.31), and after making one slight
> adjustment, it _seems_ to work fine for me under Kubuntu.
>
> But, as usual, it's late and I need to be in bed. :)  So I'd
> appreciate if folks could put additional sets of eyes on things,
> and of course test to make sure that nothing breaks when you
> try to build & package Tux Paint on your favorite platforms.
>
> Here's the one commit related to this so far:
>
>
> https://sourceforge.net/p/tuxpaint/tuxpaint/ci/6b0bc390870da4fbdd8ff3d52cb2cdc270e55c5c/
>
> --
> -bill!
> Sent from my computer
>
>
> _______________________________________________
> Tuxpaint-devel mailing list
> Tux...@li...
> https://lists.sourceforge.net/lists/listinfo/tuxpaint-devel
>