* [iptables PATCH] build: add DESTDIR to vx_bin_links
@ 2013-09-06 21:39 Laurence J. Lane
2013-09-07 0:49 ` Jan Engelhardt
0 siblings, 1 reply; 4+ messages in thread
From: Laurence J. Lane @ 2013-09-06 21:39 UTC (permalink / raw)
To: netfilter-devel; +Cc: Laurence J. Lane
Fixes dangling symlinks when using DESTDIR.
Signed-off-by: Laurence J. Lane <ljlane@debian.org>
---
iptables/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/iptables/Makefile.am b/iptables/Makefile.am
index a4246eb..ec53443 100644
--- a/iptables/Makefile.am
+++ b/iptables/Makefile.am
@@ -49,6 +49,6 @@ pkgconfig_DATA = xtables.pc
install-exec-hook:
-if test -z "${DESTDIR}"; then /sbin/ldconfig; fi;
${INSTALL} -dm0755 "${DESTDIR}${bindir}";
- for i in ${vx_bin_links}; do ${LN_S} -f "${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
+ for i in ${vx_bin_links}; do ${LN_S} -f "${DESTDIR}${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
for i in ${v4_sbin_links}; do ${LN_S} -f xtables-multi "${DESTDIR}${sbindir}/$$i"; done;
for i in ${v6_sbin_links}; do ${LN_S} -f xtables-multi "${DESTDIR}${sbindir}/$$i"; done;
--
1.8.4.rc3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [iptables PATCH] build: add DESTDIR to vx_bin_links
2013-09-06 21:39 [iptables PATCH] build: add DESTDIR to vx_bin_links Laurence J. Lane
@ 2013-09-07 0:49 ` Jan Engelhardt
2013-09-07 2:14 ` Laurence J. Lane
0 siblings, 1 reply; 4+ messages in thread
From: Jan Engelhardt @ 2013-09-07 0:49 UTC (permalink / raw)
To: Laurence J. Lane; +Cc: netfilter-devel
On Friday 2013-09-06 23:39, Laurence J. Lane wrote:
>Fixes dangling symlinks when using DESTDIR.
>
>Signed-off-by: Laurence J. Lane <ljlane@debian.org>
>- for i in ${vx_bin_links}; do ${LN_S} -f \
> "${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
>+ for i in ${vx_bin_links}; do ${LN_S} -f \
> "${DESTDIR}${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
This seems totally incorrect. By including DESTDIR in the symlink
target, you are *creating* a dangling symlink. RPM, DEB (and whatever
other packages) would suddenly contain a file that goes like
$ rpm -ql iptables
lrwxrwxrwx 1 root root 13 Jan 27 2013 /usr/sbin/iptables ->
/home/abuild/rpmbuild/BUILDROOT/iptables-1.4.20/usr/bin/xtables-multi
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [iptables PATCH] build: add DESTDIR to vx_bin_links
2013-09-07 0:49 ` Jan Engelhardt
@ 2013-09-07 2:14 ` Laurence J. Lane
2013-09-08 6:41 ` Maciej Żenczykowski
0 siblings, 1 reply; 4+ messages in thread
From: Laurence J. Lane @ 2013-09-07 2:14 UTC (permalink / raw)
To: Netfilter Development Mailinglist
You're right. Withdrawn.
On Fri, Sep 6, 2013 at 8:49 PM, Jan Engelhardt <jengelh@inai.de> wrote:
>
> On Friday 2013-09-06 23:39, Laurence J. Lane wrote:
>
>>Fixes dangling symlinks when using DESTDIR.
>>
>>Signed-off-by: Laurence J. Lane <ljlane@debian.org>
>>- for i in ${vx_bin_links}; do ${LN_S} -f \
>> "${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
>>+ for i in ${vx_bin_links}; do ${LN_S} -f \
>> "${DESTDIR}${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
>
> This seems totally incorrect. By including DESTDIR in the symlink
> target, you are *creating* a dangling symlink. RPM, DEB (and whatever
> other packages) would suddenly contain a file that goes like
>
> $ rpm -ql iptables
> lrwxrwxrwx 1 root root 13 Jan 27 2013 /usr/sbin/iptables ->
> /home/abuild/rpmbuild/BUILDROOT/iptables-1.4.20/usr/bin/xtables-multi
>
> !DSPAM:522a78438821787114786!
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [iptables PATCH] build: add DESTDIR to vx_bin_links
2013-09-07 2:14 ` Laurence J. Lane
@ 2013-09-08 6:41 ` Maciej Żenczykowski
0 siblings, 0 replies; 4+ messages in thread
From: Maciej Żenczykowski @ 2013-09-08 6:41 UTC (permalink / raw)
To: Laurence J. Lane; +Cc: Netfilter Development Mailinglist
It should probably be a relative link, but that's probably hard to do
seeing as we don't know how many levels up you need to go...
On Fri, Sep 6, 2013 at 7:14 PM, Laurence J. Lane <ljlane@debian.org> wrote:
> You're right. Withdrawn.
>
> On Fri, Sep 6, 2013 at 8:49 PM, Jan Engelhardt <jengelh@inai.de> wrote:
>>
>> On Friday 2013-09-06 23:39, Laurence J. Lane wrote:
>>
>>>Fixes dangling symlinks when using DESTDIR.
>>>
>>>Signed-off-by: Laurence J. Lane <ljlane@debian.org>
>>>- for i in ${vx_bin_links}; do ${LN_S} -f \
>>> "${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
>>>+ for i in ${vx_bin_links}; do ${LN_S} -f \
>>> "${DESTDIR}${sbindir}/xtables-multi" "${DESTDIR}${bindir}/$$i"; done;
>>
>> This seems totally incorrect. By including DESTDIR in the symlink
>> target, you are *creating* a dangling symlink. RPM, DEB (and whatever
>> other packages) would suddenly contain a file that goes like
>>
>> $ rpm -ql iptables
>> lrwxrwxrwx 1 root root 13 Jan 27 2013 /usr/sbin/iptables ->
>> /home/abuild/rpmbuild/BUILDROOT/iptables-1.4.20/usr/bin/xtables-multi
>>
>> !DSPAM:522a78438821787114786!
>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-09-08 6:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-06 21:39 [iptables PATCH] build: add DESTDIR to vx_bin_links Laurence J. Lane
2013-09-07 0:49 ` Jan Engelhardt
2013-09-07 2:14 ` Laurence J. Lane
2013-09-08 6:41 ` Maciej Żenczykowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.