Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCHv2 1/2] package/lftp: don't use host's path
Date: Sat, 27 Dec 2014 22:19:55 +0100	[thread overview]
Message-ID: <20141227211955.GN4098@free.fr> (raw)
In-Reply-To: <1419712467-20090-1-git-send-email-romain.naour@openwide.fr>

Romain, All,

On 2014-12-27 21:34 +0100, Romain Naour spake thusly:
> lftp try to link with -L/usr/lib/lftp/$(LFTP_VERSION) if
> DESTDIR is not set.
> 
> Remove useless -L option which point to a location where
> no libraries are installed.
> 
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> v2: remove -L option instead of setting DESTDIR to STAGING_DIR
>     (ThomasP)
> 
>  .../lftp/0001-Remove-unused-libraries-path.patch   | 46 ++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>  create mode 100644 package/lftp/0001-Remove-unused-libraries-path.patch
> 
> diff --git a/package/lftp/0001-Remove-unused-libraries-path.patch b/package/lftp/0001-Remove-unused-libraries-path.patch
> new file mode 100644
> index 0000000..3521ec8
> --- /dev/null
> +++ b/package/lftp/0001-Remove-unused-libraries-path.patch
> @@ -0,0 +1,46 @@
> +From 30e575f18416d67732be1e0350ee727b72b2d225 Mon Sep 17 00:00:00 2001
> +From: Romain Naour <romain.naour@openwide.fr>
> +Date: Sat, 27 Dec 2014 20:56:56 +0100
> +Subject: [PATCH] Remove unused libraries path
> +
> +When building proto modules the path to library files is
> +specified with -L$(DESTDIR)$(pkgverlibdir) which point to
> +directory that doesn't exist.
> +
> +Also, when cross-compiling and $(DESTDIR) is empty,
> +$(pkgverlibdir) contains a path to the host libraries
> +(/usr/lib/lftp/4.5.5) which is considered unsafe.
> +
> +Anyway, those -L are not needed: by the time where
> +proto modules gets built/linked, no libraries are
> +installed in this location.
> +
> +Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> +---
> + src/Makefile.am | 10 +++++-----
> + 1 file changed, 5 insertions(+), 5 deletions(-)
> +
> +diff --git a/src/Makefile.am b/src/Makefile.am
> +index d715d0a..0e221cf 100644
> +--- a/src/Makefile.am
> ++++ b/src/Makefile.am
> +@@ -59,11 +59,11 @@ liblftp_pty_la_LDFLAGS     = -avoid-version -rpath $(pkgverlibdir)
> + liblftp_network_la_LDFLAGS = -avoid-version -rpath $(pkgverlibdir) $(OPENSSL_LDFLAGS)
> + liblftp_network_la_LIBADD  = $(SOCKSLIBS) $(OPENSSL_LIBS) $(LIBGNUTLS_LIBS) $(GNULIB)
> + 
> +-proto_ftp_la_LIBADD  = -L$(DESTDIR)$(pkgverlibdir) liblftp-network.la
> +-proto_http_la_LIBADD = -L$(DESTDIR)$(pkgverlibdir) liblftp-network.la $(EXPAT_LIBS) $(ZLIB)
> +-proto_fish_la_LIBADD = -L$(DESTDIR)$(pkgverlibdir) liblftp-network.la liblftp-pty.la
> +-proto_sftp_la_LIBADD = -L$(DESTDIR)$(pkgverlibdir) liblftp-network.la liblftp-pty.la
> +-cmd_torrent_la_LIBADD  = -L$(DESTDIR)$(pkgverlibdir) liblftp-network.la
> ++proto_ftp_la_LIBADD  = liblftp-network.la
> ++proto_http_la_LIBADD = liblftp-network.la $(EXPAT_LIBS) $(ZLIB)
> ++proto_fish_la_LIBADD = liblftp-network.la liblftp-pty.la
> ++proto_sftp_la_LIBADD = liblftp-network.la liblftp-pty.la
> ++cmd_torrent_la_LIBADD  = liblftp-network.la
> + 
> + liblftp_tasks_la_SOURCES = PollVec.cc PollVec.h SMTask.cc SMTask.h ProcWait.cc\
> +  ProcWait.h GetPass.cc GetPass.h ConnectionSlot.cc ConnectionSlot.h\
> +-- 
> +1.9.3
> +
> -- 
> 1.9.3
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  parent reply	other threads:[~2014-12-27 21:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-27 20:34 [Buildroot] [PATCHv2 1/2] package/lftp: don't use host's path Romain Naour
2014-12-27 20:34 ` [Buildroot] [PATCHv2 2/2] package/tmux: remove unsafe headers path Romain Naour
2014-12-27 21:20   ` Yann E. MORIN
2014-12-27 21:45   ` Thomas Petazzoni
2014-12-27 21:19 ` Yann E. MORIN [this message]
2014-12-27 21:45 ` [Buildroot] [PATCHv2 1/2] package/lftp: don't use host's path Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141227211955.GN4098@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox