search

rfmvh / perfevents-result-collector

GNU General Public License v3.0
1 stars 1 forks source link

use os.path.join instead of dir1 + "/" + dir2 #25

Open mpavlase opened 6 years ago

mpavlase commented 6 years ago

It's more than good habit to use os.path.join instead of concatenating dirs with slashes. It's platform dependent. https://github.com/rfmvh/perfevents-result-collector/blob/dc978b2f407636dfdfaca404149342fcae047a23/tests/run_tests.py#L41

Change it in whole file.

frozenstein commented 6 years ago

I think this has been already fixed (current version v1.1).

mpavlase commented 6 years ago

Actually not, in v1.1 there is no tests directory, it's only part of develop2 branch that is not merged into master yet.