chadw75 / tunnelblick

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

Can't open .tblk under dot directory #231

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Trying to `open .foo/bar.tblk` fails. I see the attached dialog, "with .foo", 
and I see the following in Console.app:

5/16/13 1:18:13.067 PM Tunnelblick[32173]: Ignoring invalid TBReplaceIdentical 
value of '(null)' in /Users/danb/.foo/bar.tblk/Contents/Info.plist
5/16/13 1:18:13.067 PM Tunnelblick[32173]: Ignoring invalid TBSharePackage 
value of '(null)' in /Users/danb/.foo/bar.tblk/Contents/Info.plist
5/16/13 1:18:13.067 PM Tunnelblick[32173]: Must have one configuration in a 
.tblk, 0 were found in /Users/danb/.foo/bar.tblk

It succeeds if I rename .foo to foo and then `open foo/bar.tblk`, at which 
point I see the expected dialog, "with foo".

Versions:
Tunnelblick 3.3beta50 (build 3302)  -  OpenVPN 2.2.1
OSX 10.8.3

Original issue reported on code.google.com by jdanbr...@gmail.com on 16 May 2013 at 5:23

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the report and for documenting the problem so well.

I can see what's happening: Tunnelblick isn't seeing the .tblk contents because 
it isn't looking into the "invisible" folder (the folder that starts with a 
dot. Ignoring such folders was done on purpose, but I don't remember why, and I 
will rethink that decision.

This behavior may also interfere with auto-installing .tblks that are in a 
".auto-install" folder which is in the same folder as the Tunnelblick.app that 
has been double-clicked to install it.

Original comment by jkbull...@gmail.com on 16 May 2013 at 5:50

GoogleCodeExporter commented 9 years ago
The fix for this has been committed as r2343. It will be in the next release.

Thanks again for reporting this bug.

Original comment by jkbull...@gmail.com on 16 May 2013 at 11:07

GoogleCodeExporter commented 9 years ago
Awesome, thanks for the fast response.

Original comment by jdanbr...@gmail.com on 16 May 2013 at 11:09