Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not launching (Linux) #910

Closed
bjorkalx opened this issue Mar 14, 2021 · 12 comments
Closed

Not launching (Linux) #910

bjorkalx opened this issue Mar 14, 2021 · 12 comments
Assignees
Labels

Comments

@bjorkalx
Copy link

bjorkalx commented Mar 14, 2021

I have tried running from source and installing the AUR package. Python 3.8 and 3.9 gives me basically the same error.

Traceback (most recent call last):
  File "/usr/lib/edmarketconnector/./EDMarketConnector.py", line 188, in <module>
    journal_lock = JournalLock()
  File "/usr/lib/edmarketconnector/journal_lock.py", line 26, in init
    self.journal_dir_path = pathlib.Path(self.journal_dir)
  File "/usr/lib/python3.9/pathlib.py", line 1071, in new
    self = cls._from_parts(args, init=False)
  File "/usr/lib/python3.9/pathlib.py", line 696, in _from_parts
    drv, root, parts = self._parse_args(args)
  File "/usr/lib/python3.9/pathlib.py", line 680, in _parse_args
    a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not NoneType
@bjorkalx bjorkalx added bug unconfirmed An unconfirmed bug labels Mar 14, 2021
@SamathyTheManathy
Copy link

SamathyTheManathy commented Mar 15, 2021

I have a similar error, this is what it looks like in Python 3.8

This is in Ubuntu 20.04

python3 EDMarketConnector.py


Traceback (most recent call last):
  File "EDMarketConnector.py", line 186, in <module>
    journal_lock = JournalLock()
  File "/home/samtheon/Downloads/EDMarketConnector-stable/journal_lock.py", line 36, in __init__
    self.journal_dir_path = pathlib.Path(self.journal_dir)
  File "/usr/lib/python3.8/pathlib.py", line 1038, in __new__
    self = cls._from_parts(args, init=False)
  File "/usr/lib/python3.8/pathlib.py", line 679, in _from_parts
    drv, root, parts = self._parse_args(args)
  File "/usr/lib/python3.8/pathlib.py", line 663, in _parse_args
    a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not NoneType

@A-UNDERSCORE-D
Copy link
Contributor

@bjorkalx @SamathyTheManathy these fresh installs?

@A-UNDERSCORE-D
Copy link
Contributor

Also, what branch?

@SamathyTheManathy
Copy link

Freshly installed and stable.

@A-UNDERSCORE-D
Copy link
Contributor

Ah yes I see the issue. Been poking about. Develop will work for you for now, if you're happy with using it (I do daily, it should be stable.)

For me for later:
JournalLocks constructor asks for either the config defined journal location or the default one. On linux, that default is None, thus making pathlib explode. Fix that is in develop just stringifies None, so it'll probably still error down the line, but it should at least manage to start. We should probably check if the dir exists, for the develop case, but I'll look at this more tomorrow when Im likely to be more awake / ath is around to poke at

@SamathyTheManathy
Copy link

Not a problem, thanks for the quick fix!

@A-UNDERSCORE-D
Copy link
Contributor

Poke me in our discord if you still have an issue, it's a reasonably simple patch

@A-UNDERSCORE-D A-UNDERSCORE-D removed the unconfirmed An unconfirmed bug label Mar 15, 2021
@A-UNDERSCORE-D A-UNDERSCORE-D self-assigned this Mar 15, 2021
@Athanasius
Copy link
Contributor

Ah, and I was so close to fixing this when looking at macOS, except it has a default for journal_dir.

Yeah, can fix later today for 4.2.1 that we already have a few other fixes queued for.

@Athanasius
Copy link
Contributor

https://github.com/EDCD/EDMarketConnector/releases/tag/Release%2F4.2.1 is now available, also in the stable branch that should fix this.

@Athanasius
Copy link
Contributor

@bjorkalx @SamathyTheManathy Did you get chance to test 4.2.1 or later (4.2.3 is now out), or fresh pull of stable as the same code is in there now ?

@bjorkalx
Copy link
Author

@bjorkalx @SamathyTheManathy Did you get chance to test 4.2.1 or later (4.2.3 is now out), or fresh pull of stable as the same code is in there now ?

Tested 4.2.1 and 4.2.2, all good.

@SamathyTheManathy
Copy link

Sorry for the late reply, 4.2.2 works! Thanks a lot :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants