[Buildroot] [Bug 6164] New: openvpn usage of ip tool from Busybox

[Buildroot] [Bug 6164] New: openvpn usage of ip tool from Busybox

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=6164

           Summary: openvpn usage of ip tool from Busybox
           Product: buildroot
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: minor
          Priority: P5
         Component: Other
        AssignedTo: unassigned at buildroot.uclibc.org
        ReportedBy: aleksandar.zivkovic at gmail.com
                CC: buildroot at uclibc.org
   Estimated Hours: 0.0


Overview: wrong condition for openvpn configuration on ip application usage

Steps to Reproduce:
Build openvpn without selected package BR2_PACKAGE_IPROUTE2

Actual Results: 
Built application will seek for ip tool in path /bin/ip which is location of
BR2_PACKAGE_IPROUTE2. Location of Busybox version of ip is /sbin/ip

Expected Results:
If openvpn is build without selected package BR2_PACKAGE_IPROUTE2 it should
select Busybox version of ip: /sbin/ip.


Build Date & Platform: BR 2013.02

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 6164] openvpn usage of ip tool from Busybox

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=6164

--- Comment #1 from Aleksandar <aleksandar.zivkovic@gmail.com> 2013-04-04 14:54:36 UTC ---
Further mk file update solved problem on my side:

Index: openvpn.mk
===================================================================
--- openvpn.mk    (revision 12121)
+++ openvpn.mk    (revision 12122)
@@ -18,9 +18,9 @@
 endif

 ifeq ($(BR2_PACKAGE_IPROUTE2),y)
+OPENVPN_CONF_ENV += IPROUTE=/bin/ip
+else
 OPENVPN_CONF_ENV += IPROUTE=/sbin/ip
-else
-OPENVPN_CONF_ENV += IPROUTE=/bin/ip
 endif

 ifeq ($(BR2_PACKAGE_OPENVPN_LZO),y)

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 6164] openvpn usage of ip tool from Busybox

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=6164

Gustavo Zacarias <gustavo@zacarias.com.ar> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |gustavo at zacarias.com.ar

--- Comment #2 from Gustavo Zacarias <gustavo@zacarias.com.ar> 2013-04-04 16:56:53 UTC ---
*SIGH*
Actually your modification isn't 100% correct.
Busybox <1.21.x places ip in /bin, >=1.21.x places it in /sbin (like the
iproute2 package).
So some additional logic is needed.
I've just sent a patch to the list to address this, try
http://patchwork.ozlabs.org/patch/233877/

Thanks for the headsup!

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 6164] openvpn usage of ip tool from Busybox

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=6164

Gustavo Zacarias <gustavo@zacarias.com.ar> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Gustavo Zacarias <gustavo@zacarias.com.ar> 2013-05-04 13:55:59 UTC ---
Fixed in git commit 1627fb54715406d6c76461d5104ee11dabbf5870 for the upcoming
2013.05 release.

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.