On Wed, Aug 17, 2022 at 03:31:33PM -0300, Spenser Truex wrote: > On 22/08/16 10:04AM, Quentin Rameau wrote: > > Hi Spenser, > > > > > Useful since curses cocks up any portability here. We prioritise the > > > user's entries in CURSESLIB where possible, and use LIBS when pkg-config > > > cannot be used. > > > > Indeed, dealing with *ncurses* is a mess. > > Ideally, we would only link against terminfo, > > because that's what sacc needs, not curses, > > but sadly this is implemented in most system > > through ncurses. > > > > > An opinioned change (maybe the user should just figure this stuff out > > > himself or curses should be fixed). > > > > As a result, there is no good simple way to deal with that, > > and I prefer letting the user dealing with their system, > > because they should know better > > (with the potential help of documentation) > > how it is setup. > > The goal of this patch is to just "be helpful" rather than silently not > building with some opaque messages about ld. > > > Regarding the patch itself, besides the actual target idea, > > I prefer to keep the Makefile POSIX-compatible > > and you're introducing GNU make style operators. > > POSIX make doesn't appear to have $(shell) which is the main thing used > here. Instead the commands could be run inside a recipe somehow. > > (posix make documentation): https://pubs.opengroup.org/onlinepubs/9699919799/ > > > I'm not motivated to change the way we link against terminfo, > > at least not that way, > > but maybe we could introduce a comment there > > explaining that ncurses is a mess, but that already sounds like > > something most people would already know??? > > For a comment, mentioning pkg-config --libs would be sufficient. > > > In any case, thanks for your patch Spenser, > > and I hope you'll still find sacc useful, > > > > cheers! > > > > P.S.: sacc isn't really a suckless.org project, although I suppose > > people will not mind too much having it discussed there. > > > I'm satisfied with adding a comment as a solution and will submit a > patch. Then I will remember what to do the next time I compile sacc in > a couple of years. >
100% customer satisfaction guaranteed > -- > CAEE B377 FC82 BAF9 102C D22F C5CE D003 1AA8 E281 > Spenser Truex https://equwal.com -- Kind regards, Hiltjo