ActivityWatch / aw-qt

Tray icon that manages ActivityWatch processes, built with Qt.
Mozilla Public License 2.0
27 stars 29 forks source link

Unable to build a wheel #38

Closed chozabu closed 5 years ago

chozabu commented 6 years ago

pip install wheel fixed the issue, should probably be done by build script (also upgraded pip, but not sure that was needed)

ErikBjare commented 6 years ago

What wheel? What was the output? What operating system/platform?

This is not enough info to know what the issue was. Closing until more info is provided.

chozabu commented 6 years ago

Wow, bit of a quick close! But good point on the lack of information.

Sorry to say I did not log the output - but I'll see what info I can give you; OS: KDE Neon (Ubuntu 16.04)

In a little more detail: While following official "Build from source" instructions ( https://activitywatch.readthedocs.io/en/latest/installing-from-source.html ) on the step make build there were multiple error messages (something like bdist could not build wheel)

Googling got me this page: https://stackoverflow.com/questions/34819221/why-is-python-setup-py-saying-invalid-command-bdist-wheel-on-travis-ci/44862371

The suggested pip install wheel command fixed it.

This should perhaps either be listed as an OS prerequisite, or added to the list of pip packages

ErikBjare commented 6 years ago

@chozabu Sorry for the quick close, but I won't try to debug or fix something that I don't know how to replicate. I don't think this should be an issue with an upgraded pip, and while a pip install wheel should fix it I want to know why it's needed in the first place (as would probably be explained by the log).

Unless you can replicate this issue will remain closed. Thanks for creating an issue though, in case someone stumbles upon the same problem later (hopefully not).

chozabu commented 6 years ago

Hi - I tried a fresh setup to replicate

chozabu@chozabu-32:~/devel/awatchtest$ git clone --recursive https://github.com/ActivityWatch/activitywatch.git
Cloning into 'activitywatch'...
...
chozabu@chozabu-32:~/devel/awatchtest$ cd activitywatch/
chozabu@chozabu-32:~/devel/awatchtest/activitywatch$ python3 -m venv venv
chozabu@chozabu-32:~/devel/awatchtest/activitywatch$ source ./venv/bin/activate
(venv) chozabu@chozabu-32:~/devel/awatchtest/activitywatch$ make build
if [ -e "aw-core/.git" ]; then \
..
Building wheels for collected packages: peewee, strict-rfc3339, takethetime, aw-core
  Running setup.py bdist_wheel for peewee ... error
  Complete output from command /home/chozabu/devel/awatchtest/activitywatch/venv/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-td4ogh1_/peewee/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /tmp/tmp20th9uygpip-wheel- --python-tag cp35:
  /tmp/pip-build-td4ogh1_/peewee/setup.py:23: UserWarning: Cython C extensions for peewee will NOT be built, because Cython does not seem to be installed. To enable Cython C extensions, install Cython >=0.22.1.
    'extensions, install Cython >=' + cython_min_version + '.')
  usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
     or: -c --help [cmd1 cmd2 ...]
     or: -c --help-commands
     or: -c cmd --help

  error: invalid command 'bdist_wheel'

  ----------------------------------------
  Failed building wheel for peewee
  Running setup.py clean for peewee
  Running setup.py bdist_wheel for strict-rfc3339 ... error
  Complete output from command /home/chozabu/devel/awatchtest/activitywatch/venv/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-td4ogh1_/strict-rfc3339/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /tmp/tmp80l1tlg_pip-wheel- --python-tag cp35:
  usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
     or: -c --help [cmd1 cmd2 ...]
     or: -c --help-commands
     or: -c cmd --help

  error: invalid command 'bdist_wheel'

  ----------------------------------------
  Failed building wheel for strict-rfc3339
  Running setup.py clean for strict-rfc3339
  Running setup.py bdist_wheel for takethetime ... error
  Complete output from command /home/chozabu/devel/awatchtest/activitywatch/venv/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-td4ogh1_/takethetime/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /tmp/tmp_zgnorkjpip-wheel- --python-tag cp35:
  usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
     or: -c --help [cmd1 cmd2 ...]
     or: -c --help-commands
     or: -c cmd --help

  error: invalid command 'bdist_wheel'

  ----------------------------------------
  Failed building wheel for takethetime
  Running setup.py clean for takethetime
  Running setup.py bdist_wheel for aw-core ... error
  Complete output from command /home/chozabu/devel/awatchtest/activitywatch/venv/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-tftk8_z2-build/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /tmp/tmpvmlei237pip-wheel- --python-tag cp35:

trying out just upgrading pip seems to fix things

(venv) chozabu@chozabu-32:~/devel/awatchtest/activitywatch$ pip3 install --upgrade pip
(venv) chozabu@chozabu-32:~/devel/awatchtest/activitywatch$ make build
...
OK
...
python3 -c "import aw_server; print(aw_server.__version__)"
v0.8-dev-5a6af72
(venv) chozabu@chozabu-32:~/devel/awatchtest/activitywatch$

Perhaps upgrading pip could be mentioned in the build guide?

ErikBjare commented 6 years ago

@chozabu Nice! Reopening