Superbil / tunnelblick

Automatically exported from code.google.com/p/tunnelblick
1 stars 0 forks source link

TunnelBlick needs to update the CFBundleVersion string in the kext Info.plist files to correspond to the overall version #191

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
TunnelBlick has two kexts, tun.kext and tap.kext.  In each case, the 
CFBundleVersion in the Info.plist file is set to 1.0, e.g.
        <key>CFBundleVersion</key>
        <string>1.0</string>
This should be set to match the overall version of TunnelBlick.  Without doing 
so, it's impossible to distinguish panics from different versions of the kexts, 
since a panic log shows the version from the CFBundleVersion field.

      Kernel Extensions in backtrace:
         net.tunnelblick.tun(1.0)[576DEDE7-B6C1-610E-B7E4-C54DBDA631AF]@0xffffff7f80794000->0xffffff7f8079bfff

This should be updated with each build.

Original issue reported on code.google.com by brian.be...@gmail.com on 11 Aug 2011 at 4:14

GoogleCodeExporter commented 9 years ago
Thanks for your report. This has been fixed in the trunk as r1744 and will be 
included in the next release.

This was implemented by changing the version number in the tun and tap kexts to 
include the tuntap version number and the Tunnelblick build number.

The Tunnelblick build number was also added to the tun and tap Info.plist 
CFBundleShortVersionString entries.

Example of the new entries after building from r1744:

     CFBundleVersion = "20090913.2744"

     CFBundleShortVersionString = "20090913 (Tunnelblick build 2744)"

Original comment by jkbull...@gmail.com on 13 Aug 2011 at 2:55

GoogleCodeExporter commented 9 years ago
Arrgghh. r1745 fixes a problem caused by r1744: CFBundleVersion can't have such 
a large, complex number. Now it contains the same thing for the tun and tap 
kexts as it does for Tunnelblick.app: the build number.

So in a build from r1745:

     CFBundleVersion = "2745"

     CFBundleShortVersionString = "20090913 (Tunnelblick build 2745)"

Original comment by jkbull...@gmail.com on 13 Aug 2011 at 5:57