-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 2011-10-30 22:41, Hans-Christoph Steiner wrote: > > Having the name SHARED_SOURCE singular highlights that its a single file, > which I think it should be. But since its an internal variable, I would be > OK with it having a different name.
i really cannot understand, why SHARED_SOURCE _should_ only be a single file. do you have any reference that claims that single-file libraries are somewhat "superior" (e.g. less confusing to the casual onlooker)? anyhow... > > I don't see a gain to making it multiple files, unless you are doing > something very complicated, in which case, you'll need to customize the > Makefile. At that point, the Makefile is easy enough to read for someone who > does Makefiles, so they don't need it documented. > >> - - standardize the shared library name to lib$(LIBNAME).$(EXT) >> >> that is all. >> > It already is: > > SHARED_SOURCE ?= $(wildcard lib$(LIBRARY_NAME).c) > SHARED_HEADER ?= $(shell test ! -e $(LIBRARY_NAME).h || echo > $(LIBRARY_NAME).h) > SHARED_LIB = $(SHARED_SOURCE:.c=.$(SHARED_EXTENSION)) not really. SHARED_LIB is generated by replacing all ".c" extension by ".$(SHARED_EXTENSION) within SHARED_SOURCE, which is a space delimited list of tokens. your SHARED_SOURCE happens to consist only of a single token, but at the same time it is settable by the user (and indeed documents that it could be overridden by the use of "?=") whereas LIBRARY_NAME is documented to be a single name (well i thought so; i cannot find any reference to this either) if we leave variable names as they are, my proposed change is a one-liner (see attachment) > SHARED_TCL_LIB = $(wildcard lib$(LIBRARY_NAME).tcl) does this already do anything? it seems like the only use of SHARED_TCL_LIB is within the "showsetup" target. > I figured you'd want something more complicated, so I put in the ?= for you. > Like before with ALL_*, take the template and modify it for your library. If > its implemented in a way that still meets the goal of a simple interface, > I'll merge it in. ah, we should create some code for me to say "this is _really_ a simple change" mfgasdr IOhannes -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk6uYhQACgkQkX2Xpv6ydvQXKwCg8aASpKApdG7MLDffKHtlrWF8 JTQAoJ7CrewiC49OnefL2oKyL9bsCM34 =yPTG -----END PGP SIGNATURE-----
--- Makefile.template 2011-10-31 09:36:43.000000000 +0100 +++ Makefile 2011-10-31 09:41:03.000000000 +0100 @@ -234,7 +234,7 @@ SHARED_SOURCE ?= $(wildcard lib$(LIBRARY_NAME).c) SHARED_HEADER ?= $(shell test ! -e $(LIBRARY_NAME).h || echo $(LIBRARY_NAME).h) -SHARED_LIB = $(SHARED_SOURCE:.c=.$(SHARED_EXTENSION)) +SHARED_LIB ?= lib$(LIBRARY_NAME:=.$(SHARED_EXTENSION)) SHARED_TCL_LIB = $(wildcard lib$(LIBRARY_NAME).tcl) .PHONY = install libdir_install single_install install-doc install-examples install-manual clean distclean dist etags $(LIBRARY_NAME)
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ Pd-dev mailing list [email protected] http://lists.puredata.info/listinfo/pd-dev
