[Buildroot] [PATCH] netkitftp: added package with Makefile.autotools.in support
Hugues
hugues at hiegel.fr
Tue Oct 27 09:35:07 UTC 2009
Hello, you ;-)
Ce cher Lionel Landwerlin <llandwerlin at gmail.com> a dit :
> Le lundi 26 octobre 2009 à 22:15 +0100, Hugues a écrit :
>> Hello,
>
>> > Hugues> diff --git a/package/netkitftp/netkitftp-configure.patch b/package/netkitftp/netkitftp-configure.patch
>> >
>> > That one as well.
>>
>> Well... This one was already packaged with the package I've got, I don't
>> have any idea why it is needed. I'll check this and then verify if it is
>> really needed (hope it is not :-) )
>
> It prevents the launch of cross compiled binaries which would make exit
> the configure script.
Thanks for the check !
>> > Hugues> +++ b/package/netkitftp/netkitftp.mk
>> > Hugues> @@ -0,0 +1,23 @@
>> > Hugues> +#############################################################
>> > Hugues> +#
>> > Hugues> +# netkitftp
>> > Hugues> +#
>> > Hugues> +#############################################################
>> > Hugues> +NETKITFTP_VERSION:=0.17
>> > Hugues> +NETKITFTP_SOURCE:=netkit-ftp_$(NETKITFTP_VERSION).orig.tar.gz
>> > Hugues> +NETKITFTP_PATCH:=netkit-ftp_0.17-16.diff.gz
>> > Hugues> +NETKITFTP_SITE:=ftp://ftp.debian.org/debian/pool/main/n/netkit-ftp/
>> >
>> > Is that a stable URL? Would something on snapshot.debian.net be better?
>> > We have BR2_DEBIAN_MIRROR, which should be used.
>>
>> Oh, that's right !
>> I'll correct this.
>>
>> > Hugues> +
>> > Hugues> +NETKITFTP_CONF_OPT = --installroot=$(TARGET_DIR) \
>> >
>> > Something regarding TARGET_DIR at configure time? Doesn't that leak
>> > paths on the build host to the installed binaries?
>>
>> Well, I didn't investigated further, maybe is this not needed. I'll try
>> without and throw then this out, if not.
>> Or comment it, if needed :o)
>
> That package's makefile/configure is definitely strange...
Yeah, isn't it ?
> I just gave it a look and it seems INSTALL_PREFIX is not handled
> anywhere in the package.
> So you can probably drop it from the NETKITFTP_INSTALL_TARGET_OPT
> variable.
Mhmm, that seems correct.
>> > That uninstall would also remove all other man pages, so
>> > don't do that.
>>
>> Mhmm... I'm not sure I understood this.. Why do you talk about uninstall ?
>> The netkitftp's 'install' rule does installs the manpages with the
>> binary, without ability to call just the install of the binary without
>> the manpages.
>> So I did like I see in other packages : just delete the
>> $(TARGET_DIR)/usr/man folder... Is this a bad idea ?
>> What do you suggest at this stage ?
>> Doing nothing, this will be cleaned later ?
>
> Yeah, I think what Peter wanted to tell is that man pages cleaning as
> well as locales cleaning is handled globaly by buildroot that the end of
> the generation process and depending on your config file.
Ok, right, I just read his answer.
> See top Makefile :
>
> ifeq ($(BR2_ENABLE_LOCALE_PURGE),y)
> TARGETS+=target-purgelocales
> endif
>
> We probably have to make a pass on all unmaintained/old packages to fix
> that.
And that doesn't look like just an easy job ;-)
Thanks for your comments, dude,
--
Hugues Hiegel [http://www.hiegel.fr/~hugues/]
More information about the buildroot
mailing list