David Wood wrote:
The above patch is tested and working against 1.1.5. Passing
--without-docdir to configure generates a warning and prevents the
installation of any documentation.
If, after any desired tidying up, this can be committed to the CVS for
branch_1_1 and HEAD I would be
In message [EMAIL PROTECTED], Alan DeKok
[EMAIL PROTECTED] writes
Nicolas Baradakis wrote:
...
+ifeq ($(docdir),no)
+install:
+@echo 'Documentation files will NOT be installed.'
+else
I'd prefer not to have multiple install targets... even if they're
wrapped with 'ifeq'. Sort of like
David Wood wrote:
In message [EMAIL PROTECTED], Nicolas
Baradakis [EMAIL PROTECTED] writes
David Wood wrote:
There's two other things I'm currently modifying the source to achieve,
on which I'd appreciate comments.
patch-doc::Makefile surrounds the contents of the install target in
Nicolas Baradakis wrote:
...
+ifeq ($(docdir),no)
+install:
+ @echo 'Documentation files will NOT be installed.'
+else
I'd prefer not to have multiple install targets... even if they're
wrapped with 'ifeq'. Sort of like how #ifdef's in the code should be
avoided.
Maybe instead:
Hi Nicolas, Alan and everyone,
In message [EMAIL PROTECTED], Nicolas
Baradakis [EMAIL PROTECTED] writes
David Wood wrote:
The second group of patches that remain I'm going to post below -
because I think they might be candidates for committing to FreeRADIUS
itself. It was clearly felt by
David Wood wrote:
The second group of patches that remain I'm going to post below -
because I think they might be candidates for committing to FreeRADIUS
itself. It was clearly felt by previous contributors to the port that
the Makefiles don't always contain $(CFLAGS) when they would be
Nicolas Baradakis wrote:
..
Every downstream distribution has its own mechanism to handle the
config files. I don't know the opinion of the others, but I'm unsure
about a configure option that's useful for the FreeBSD port only.
It's easier to run a port-specific sed script.
i.e. This is
7 matches
Mail list logo