[MacPorts] #16392: vtun 3.0.2 patch fix
#16392: vtun 3.0.2 patch fix --------------------------------+------------------------------------------- Reporter: emertens@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.6.0 Keywords: | --------------------------------+------------------------------------------- The patch included in vtun 3.0.2 needs to be updated to the following (the change is the md5.h line). This is required to build. --- lfd_legacy_encrypt.c 2008-07-09 16:06:01.000000000 -0400 +++ lfd_legacy_encrypt.c 2008-07-09 16:07:06.000000000 -0400 @@ -55,7 +55,7 @@ #include <openssl/blowfish.h> #else /* YAY - We're MAC OS */ -#include <sys/md5.h> -#include <crypto/blowfish.h> +#include <openssl/md5.h> +#include <openssl/blowfish.h> #endif /* __APPLE_CC__ */ #define ENC_BUF_SIZE VTUN_FRAME_SIZE + 16 -- Ticket URL: <http://trac.macports.org/ticket/16392> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Changes (by ryandesign@macports.org): * cc: ryandesign@macports.org (added) * milestone: => Port Bugs Comment: Let me make sure I understand. As shipped, vtun 3.0.2 has this set of definitions in lfd_legacy_encrypt.c: {{{ #ifndef __APPLE_CC__ /* OpenSSL includes */ #include <openssl/md5.h> #include <openssl/blowfish.h> #else /* YAY - We're MAC OS */ #include <sys/md5.h> #include <crypto/blowfish.h> #endif /* __APPLE_CC__ */ }}} In other words, on non-Mac OS X systems, it uses OpenSSL includes for md5 and blowfish, and on Mac OS X it uses includes built into the operating system. The vtun 3.0.2_0 port has a [browser:trunk/dports/net/vtun/files/patch- lfd_legacy_encrypt.c.diff@38368 patch] which makes the software always use the OpenSSL blowfish include, even on Mac OS X. I do not know why this was done; it has been there since the port was initially added in r6236. Now you are proposing that the md5 include should also be taken from OpenSSL, even on Mac OS X, effectively reducing the above definitions to this: {{{ /* OpenSSL includes */ #include <openssl/md5.h> #include <openssl/blowfish.h> }}} You have filed this ticket as a defect. What problem are you observing as a result of these includes -- or, if not a problem, what is gained by changing the includes? How can I reproduce the issue on my system so that I can confirm that your change is the correct fix? The port builds fine for me with or without your change, on Mac OS X 10.4.11 on Intel with Xcode 2.5. What version of Mac OS X and Xcode do you have, and on which processor architecture? -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment(by emertens@gmail.com): My change was required to build this port on OS X 10.5 on x86 -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ---------------------------------+------------------------------------------ Comment(by emertens@gmail.com): My change was required to build this port on OS X 10.5 on x86 with XCode 3.1 -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ticketformatting Port: | ---------------------------------+------------------------------------------ Changes (by ryandesign@macports.org): * keywords: => ticketformatting -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: ryandesign@macports.org Type: defect | Status: assigned Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ticketformatting Port: | ---------------------------------+------------------------------------------ Changes (by ryandesign@macports.org): * cc: ryandesign@macports.org (removed) * owner: macports-tickets@lists.macosforge.org => ryandesign@macports.org * status: new => assigned Comment: Ok, yes, I see, on Mac OS X 10.5.4 with Xcode 3.1 on a Power Mac G4 I get {{{ /usr/bin/gcc-4.0 -O2 -I/mp/include -I/mp/include -I/usr/include/openssl -I/usr/include/openssl -I/usr/include/openssl -I/usr/include/openssl -DVTUN_CONFIG_FILE=\"/mp/etc/vtund.conf\" -DVTUN_PID_FILE=\"/mp/var/run/vtund.pid\" -DVTUN_STAT_DIR=\"/mp/var/log/vtund\" -DVTUN_LOCK_DIR=\"/mp/var/lock/vtund\" -c lfd_legacy_encrypt.c lfd_legacy_encrypt.c:57:21: error: sys/md5.h: No such file or directory }}} Now let me try it with the patch. -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: ryandesign@macports.org Type: defect | Status: closed Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: fixed | Keywords: ticketformatting Port: | ---------------------------------+------------------------------------------ Changes (by ryandesign@macports.org): * status: assigned => closed * resolution: => fixed Comment: Yup, patch fixes it. Applied a fix in r39861. Thanks! -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16392: vtun 3.0.2 patch fix ---------------------------------+------------------------------------------ Reporter: emertens@gmail.com | Owner: ryandesign@macports.org Type: defect | Status: closed Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: fixed | Keywords: Port: vtun | ---------------------------------+------------------------------------------ Changes (by jmr@macports.org): * keywords: ticketformatting => * port: => vtun Old description:
The patch included in vtun 3.0.2 needs to be updated to the following (the change is the md5.h line). This is required to build.
--- lfd_legacy_encrypt.c 2008-07-09 16:06:01.000000000 -0400 +++ lfd_legacy_encrypt.c 2008-07-09 16:07:06.000000000 -0400 @@ -55,7 +55,7 @@ #include <openssl/blowfish.h> #else /* YAY - We're MAC OS */ -#include <sys/md5.h> -#include <crypto/blowfish.h> +#include <openssl/md5.h> +#include <openssl/blowfish.h> #endif /* __APPLE_CC__ */
#define ENC_BUF_SIZE VTUN_FRAME_SIZE + 16
New description: The patch included in vtun 3.0.2 needs to be updated to the following (the change is the md5.h line). This is required to build. {{{ --- lfd_legacy_encrypt.c 2008-07-09 16:06:01.000000000 -0400 +++ lfd_legacy_encrypt.c 2008-07-09 16:07:06.000000000 -0400 @@ -55,7 +55,7 @@ #include <openssl/blowfish.h> #else /* YAY - We're MAC OS */ -#include <sys/md5.h> -#include <crypto/blowfish.h> +#include <openssl/md5.h> +#include <openssl/blowfish.h> #endif /* __APPLE_CC__ */ #define ENC_BUF_SIZE VTUN_FRAME_SIZE + 16 }}} -- -- Ticket URL: <http://trac.macports.org/ticket/16392#comment:7> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts