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

[Bill K. <nb...@so...>]

On Wed, Jun 07, 2023 at 10:10:35AM +0100, Will Thompson wrote:
> 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.

Yay!


> 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.

Sheesh, I dunno...  I guess it was probably my idea (or maybe someone
suggested it to me?) to use the XDG stuff to install all the icons.
It seemed like a smart move at the time... I'm surprised it turned out
to be such an annoyance.

The whole Makefile (like the whole codebase) needs a refactoring, I'm sure.
There's been a lot of additions & removals, in terms of requirements
(Pango & SVG stuff was awkward for a long time) and supported platforms
(Maemo & XO-1 no longer seem to be relevant?)

Only so many hours in the day, so I focus first on stuff that brings me joy.
(That's why we have things like "Fix XOR bug with blinking text cursor"
ticket from 2004. <anguish> ;-) )

-- 
-bill!
Sent from my computer