On Tue, 10 Apr 2007, Barry deFreese wrote:

> Hey folks,
> 
> Here is what I did to build ntp 4.2.2.  Let me know what you think.

I believe the patch could be a little bit smaller.

> +ifeq ($(DEB_HOST_ARCH_OS),hurd)
> +   ./configure CFLAGS='$(CFLAGS)' \
> +       --prefix=/usr \
> +       --enable-all-clocks --enable-parse-clocks --enable-SHM \
> +       --disable-debugging --sysconfdir=/var/lib/ntp \
> +       --with-sntp=no \
> +       --enable-linuxcaps \
> +       --enable-ipv6=no \
> +       --disable-dependency-tracking
> +else
>    ./configure CFLAGS='$(CFLAGS)' \
>        --prefix=/usr \
>        --enable-all-clocks --enable-parse-clocks --enable-SHM \
> @@ -25,6 +37,7 @@
>        --with-sntp=no \
>        --enable-linuxcaps \
>        --disable-dependency-tracking
> +endif

Intead of that, I would (previoulsy) use DEB_HOST_ARCH_OS to define
IPV6FLAGS appropriately (or not) and then use something like this:

        ./configure CFLAGS='$(CFLAGS)' $(IPV6FLAGS) \
                 --prefix=/usr \
        etc etc

> +ifeq ($(DEB_HOST_ARCH_OS),hurd)
> +   for file in ntpdate ntp-wait ntpd tickadj ntp-keygen; do \
> +       mv debian/ntp/usr/bin/$$file debian/ntp/usr/sbin/$$file || exit; \
> +   done
> +else
>    for file in ntpdate ntp-wait ntpd ntptime tickadj ntp-keygen; do \
>        mv debian/ntp/usr/bin/$$file debian/ntp/usr/sbin/$$file || exit; \
>    done
> +endif

Same here, you could define a suitable variable first, and then use
the same "for":

        for file in $(executables); do
                etc.
        done


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to