-
Notifications
You must be signed in to change notification settings - Fork 21
Open
Labels
Description
it would be nice if pd-lib-builder recognized IRIX as a system.
it's very similar to the "Linux" system we already have, except that:
- the latest(?, at least the one commonly found on those systems) gcc is GCC-3.3
- for calculating the dependencies we might need to provide special flags to the compiler.
- sometimes we need specific compiler flags for a given build, which the user can provide with
CPPFLAGS,CFLAGSandCXXFLAGS; however these flags are not used for thedepcheck:pd-lib-builder/Makefile.pdlibbuilder
Lines 667 to 668 in 7752526
# flags for dependency checking (cflags from makefile may define -I options) depcheck.flags := $(cpp.flags) $(cflags) - we could use
cflags, but this is not a user-facing variable, and passing it on the cmdline (e.g.make cflags=-std=c99), will completely override anycflagsset in the library specificMakefile☹️ - we could also set the
cflagsin the library specificMakefile, but that doesn't really work if the flags are very specific to the compiler on the system☹️ - so ideally we would be able to inject some (additional)
depcheckflags
- sometimes we need specific compiler flags for a given build, which the user can provide with
- the default file extension (
.pd_irix6or.pd_irix5) - the
warn.flagsinclude flags that are not yet available (pd-lib-builder/Makefile.pdlibbuilder
Line 655 in 7752526
warn.flags = -Wall -Wextra -Wshadow -Winline -Wstrict-aliasing -Wextra) and that seem to trigger on false positives (-Winline)
since I'm not entirely convinced whether adding support for such a niche system is actually desirable/worth it, I'll try to see if I can make it work with simply passing the right vars to make (in which case I guess we can consider this feature request moot)
Reactions are currently unavailable