Is hard coding the relative path in that way frowned apon?
Most certainly, especially because you have to account for differences in path format between operating systems.
I'd recommend incorporating pkg_resources into your approach above.
pkg_resources
def conf(): config = configparser.ConfigParser(converters={'list': lambda x: [i.strip() for i in x.split(',')]}, allow_no_value=True) path = pkg_resources.resource_filename( __name__, os.path.join(os.pardir, 'Resources', 'generateandsend.ini') ) config.read(path) section = config['test'] string_a = section.get('StringA', None) string_b = section.get('StringB', None) return string_a, string_b
I believe that will work? You'll have to check how config.read() handles an absolute path.
config.read()
Beautiful. Thanks. I had to massage it a little and remove os.pardir, as it was giving me a false directory on my windows machine (C:\tmp\generateandsend\..\Resources\generateandsend.ini).
os.pardir
C:\tmp\generateandsend\..\Resources\generateandsend.ini
The resultant path variable now looks like:
path = pkg_resources.resource_filename(__name__, os.path.join('Resources', 'dummy.ini'))
I just need to test this on my Linux box
Cheers again. Send the bill to...... 😉
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Most certainly, especially because you have to account for differences in path format between operating systems.
I'd recommend incorporating
pkg_resourcesinto your approach above.I believe that will work? You'll have to check how
config.read()handles an absolute path.Beautiful. Thanks. I had to massage it a little and remove
os.pardir, as it was giving me a false directory on my windows machine (C:\tmp\generateandsend\..\Resources\generateandsend.ini).The resultant path variable now looks like:
I just need to test this on my Linux box
Cheers again. Send the bill to...... 😉