joesox / pswrdgen

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

Cross Platform improvements #9

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Need to use os.path.join for file paths.Also need to use sys.platform for 
windows specific processing.  For example:

if sys.platform == 'Win32':
    FS_ROOT = 'C:'
else:
    FS_ROOT = '/'

WORDNETPATH=os.path.join( FS_ROOT, 'WordNet', '2.1', 'dict' )

-Suggested by GregC.

Original issue reported on code.google.com by joesox on 21 Aug 2007 at 1:44

GoogleCodeExporter commented 9 years ago
A step forwards, although for file opening python should quietly convert 
between /
and \. The bigger problem is that the folder (at least on Mac OS X and probably 
on
Unix is /usr/local/WordNet-X.Y/dict/ not /Wordnet/X.Y/dict. Also the current 
version
of wordnet for Unix is 3.0 not 2.1 .

Original comment by edd...@googlemail.com on 21 Aug 2007 at 3:06

GoogleCodeExporter commented 9 years ago
Thanks for your suggestions. Version 0.2.0 is posted and we are asking for Non-
Windows testers.  Thanks!

Original comment by joesox on 31 Aug 2007 at 3:58

GoogleCodeExporter commented 9 years ago
Tested and works on Mac OS X. Have moved some effectively repeated lines out of 
the
system specific code. This needs testing on other OS's but works on mine and 
*should*
be fine anywhere the old code works.

Original comment by edd...@googlemail.com on 3 Sep 2007 at 2:45

GoogleCodeExporter commented 9 years ago

Original comment by joesox on 16 Sep 2007 at 1:35