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

mail-client/alot: Add version bump 0.9.1 #16680

Conversation

GuillaumeSeren
Copy link
Contributor

No description provided.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @GuillaumeSeren
Areas affected: ebuilds
Packages affected: dev-python/urwidtrees, mail-client/alot

dev-python/urwidtrees: @gentoo/proxy-maint (maintainer needed)
mail-client/alot: @GuillaumeSeren, @gentoo/proxy-maint

Linked bugs

Bugs linked: 732360, 732362


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added maintainer-needed There is at least one affected package with no maintainer. Review it if you can. assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jul 12, 2020
# Copyright 1999-2020 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2

EAPI=7
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add an empty line below

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

# Distributed under the terms of the GNU General Public License v2

EAPI=7
PYTHON_COMPAT=( python{3_6,3_7,3_8} )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you could put 3_ outside of {}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@@ -18,17 +19,16 @@ IUSE="doc test"
RESTRICT="!test? ( test )"

RDEPEND="
>=app-crypt/gpgme-1.9.0[python,${PYTHON_USEDEP}]
>=dev-python/configobj-4.7.0[${PYTHON_USEDEP}]
app-crypt/gpgme[python,${PYTHON_USEDEP}]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why did you remove >=?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just because the minimal available version in portage is sufficient (1.13.0).

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-12 17:26 UTC
Newest commit scanned: a62a60a
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/87d95bf5b2/output.html

LICENSE="GPL-3"
SLOT="0"
KEYWORDS="~amd64 ~x86"
IUSE="doc"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should probably use distutils_enable_sphinx, which appends the doc keyword to IUSE for you: you can drop this line.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Comment on lines 33 to 45
src_compile() {
distutils-r1_src_compile

use doc && emake -C docs html
}

src_install() {
distutils-r1_src_install

if use doc; then
dodoc -r docs/build/html/.
fi
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those lines can be replaced by a single distutils_enable_sphinx call

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

>=dev-python/twisted-18.4[${PYTHON_USEDEP}]
net-mail/mailbase
>=net-mail/notmuch-0.27[crypt,python]
net-mail/notmuch[crypt,python]
"
DEPEND="
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be BDEPEND. It can be called after distutils_enable_tests, which automatically appends test? ( ${RDEPEND} ) to it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok.

"
DEPEND="
dev-python/setuptools[${PYTHON_USEDEP}]
doc? ( dev-python/sphinx[${PYTHON_USEDEP}] )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop this and use distutils_enable_sphinx

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed on irc this helper does not work when building multiple (here man page),
sphinx target.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should put this as a comment in the ebuild so that one knows why distutils_enable_sphinx is not used

"
DEPEND="
dev-python/setuptools[${PYTHON_USEDEP}]
doc? ( dev-python/sphinx[${PYTHON_USEDEP}] )
test? (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

append mock to BDEPEND after calling distutils_enable_tests

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not get it at first now it is ok, thank you.

Comment on lines 45 to 52
python_compile_all() {
emake -C docs man
use doc && emake -C docs html
}

python_install_all() {
use doc && local HTML_DOCS=( docs/build/html/. )
doman docs/build/man/*
distutils-r1_python_install_all

insinto /usr/share/alot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These lines can be replaced by a call to distutils_enable_sphinx

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately no.

@@ -3,7 +3,8 @@

EAPI=7

PYTHON_COMPAT=( python3_{6,7} )
DISTUTILS_USE_SETUPTOOLS=rdepend
PYTHON_COMPAT=( python3_{6,7,8} )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests fail with py3.8 because notmuch does not have py3.8 yet. This is why the RESTRICT below?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nevermind: #16668 will add py3.8 to notmuch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool

@GuillaumeSeren GuillaumeSeren force-pushed the mail-client/alot-add-version-bump-0.9.1 branch from a62a60a to 33e3f21 Compare July 19, 2020 12:48
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-19 13:11 UTC
Newest commit scanned: 33e3f21
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/6aef654759/output.html

@GuillaumeSeren GuillaumeSeren force-pushed the mail-client/alot-add-version-bump-0.9.1 branch from 33e3f21 to f2f5c53 Compare July 24, 2020 20:46
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-24 21:51 UTC
Newest commit scanned: f2f5c53
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/18b6318c4a/output.html

Comment on lines +40 to +41
BDEPEND+=" test? ( dev-python/mock[${PYTHON_USEDEP}] ) "

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests.widgets.test_globals (unittest.loader._FailedTest) ... ERROR

======================================================================
ERROR: alot.buffers (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: alot.buffers
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 462, in _find_test_path
    package = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/__init__.py", line 7, in <module>
    from .envelope import EnvelopeBuffer
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/envelope.py", line 9, in <module>
    from ..widgets.globals import HeadersList
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: alot.db (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: alot.db
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 462, in _find_test_path
    package = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.commands.test_envelope (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.commands.test_envelope
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/commands/test_envelope.py", line 26, in <module>
    from alot.commands import envelope
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/envelope.py", line 20, in <module>
    from . import globals
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/globals.py", line 24, in <module>
    from .. import buffers
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/__init__.py", line 7, in <module>
    from .envelope import EnvelopeBuffer
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/envelope.py", line 9, in <module>
    from ..widgets.globals import HeadersList
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.commands.test_global (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.commands.test_global
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/commands/test_global.py", line 24, in <module>
    from alot.commands import globals as g_commands
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/globals.py", line 24, in <module>
    from .. import buffers
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/__init__.py", line 7, in <module>
    from .envelope import EnvelopeBuffer
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/envelope.py", line 9, in <module>
    from ..widgets.globals import HeadersList
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.commands.test_init (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.commands.test_init
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/commands/test_init.py", line 10, in <module>
    from alot.commands import thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/thread.py", line 20, in <module>
    from .globals import ExternalCommand
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/globals.py", line 24, in <module>
    from .. import buffers
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/__init__.py", line 7, in <module>
    from .envelope import EnvelopeBuffer
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/envelope.py", line 9, in <module>
    from ..widgets.globals import HeadersList
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.commands.test_thread (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.commands.test_thread
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/commands/test_thread.py", line 11, in <module>
    from alot.commands import thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/thread.py", line 20, in <module>
    from .globals import ExternalCommand
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/commands/globals.py", line 24, in <module>
    from .. import buffers
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/__init__.py", line 7, in <module>
    from .envelope import EnvelopeBuffer
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/buffers/envelope.py", line 9, in <module>
    from ..widgets.globals import HeadersList
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.db.test_envelope (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.db.test_envelope
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/db/test_envelope.py", line 25, in <module>
    from alot.db import envelope
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.db.test_manager (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.db.test_manager
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/db/test_manager.py", line 12, in <module>
    from alot.db.manager import DBManager
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.db.test_message (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.db.test_message
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/db/test_message.py", line 21, in <module>
    from alot.db import message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.db.test_thread (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.db.test_thread
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/db/test_thread.py", line 22, in <module>
    from alot.db import thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.db.test_utils (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.db.test_utils
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/db/test_utils.py", line 26, in <module>
    from alot.db import utils
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.test_completion (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.test_completion
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/test_completion.py", line 10, in <module>
    from alot.completion.abooks import AbooksCompleter
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/completion/abooks.py", line 7, in <module>
    from ..db.utils import formataddr
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


======================================================================
ERROR: tests.widgets.test_globals (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: tests.widgets.test_globals
Traceback (most recent call last):
  File "/usr/lib64/python3.6/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.6/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/tests/widgets/test_globals.py", line 22, in <module>
    from alot.widgets import globals as globals_
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/widgets/globals.py", line 14, in <module>
    from ..db.attachment import Attachment
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/__init__.py", line 4, in <module>
    from .thread import Thread
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/thread.py", line 6, in <module>
    from .message import Message
  File "/var/tmp/portage/mail-client/alot-0.9.1/work/alot-0.9.1/alot/db/message.py", line 10, in <module>
    from notmuch import NullPointerError
ModuleNotFoundError: No module named 'notmuch'


----------------------------------------------------------------------
Ran 190 tests in 0.631s

So you'll have to add it here. Most likely to RDEPEND too.

dev-python/urwidtrees[${PYTHON_USEDEP}]
>=dev-python/twisted-18.4[${PYTHON_USEDEP}]
net-mail/mailbase
net-mail/notmuch[crypt,python]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think you'll need to add PYTHON_USEDEP here, the test phase clearly implies that. However with notmuch-30 I'm not sure how it affects the runtime, you probably have a better idea? How is notmuch used by alot? It should be broken with 30 currently. Maybe the version can be restricted for now, or we wait until notmuch-30 has been fixed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think you'll need to add PYTHON_USEDEP here, the test phase clearly implies that. However with notmuch-30 I'm not sure how it affects the runtime, you probably have a better idea?

I am not really sure, what I can say alot use the python binding provided by notmuch,
I used to think that when notmuch is installed it is build with all python version, maybe I am wrong.

Maybe the version can be restricted for now, or we wait until notmuch-30 has been fixed?
Yes I contacted upstream (pazz/alot#1539) it seems,
a PR is already going on to port to 'notmuch2' module (pazz/alot#1511).

So with all that, I think I probably fix to '<=net-mail/notmuch-0.30[crypt,python,${PYTHON_USEDEP}]',
until everything is fixed.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-08-07 10:19 UTC
Newest commit scanned: bee4e2b
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/4c9d002b3d/output.html

@GuillaumeSeren GuillaumeSeren force-pushed the mail-client/alot-add-version-bump-0.9.1 branch 2 times, most recently from 891a691 to 09f891e Compare August 7, 2020 10:32
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-08-07 10:44 UTC
Newest commit scanned: 891a691
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/15d2eb3e51/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-08-07 11:04 UTC
Newest commit scanned: 09f891e
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/48f38904bb/output.html

Closes: https://bugs.gentoo.org/732360
Package-Manager: Portage-2.3.99, Repoman-2.3.23
Signed-off-by: Guillaume Seren <[email protected]>
@GuillaumeSeren GuillaumeSeren force-pushed the mail-client/alot-add-version-bump-0.9.1 branch from 09f891e to 41c8b7e Compare August 8, 2020 17:11
@GuillaumeSeren
Copy link
Contributor Author

Hey,
I have fixed my chroot test setup, and I detected one new issue (urwid).

I tested it against a stage3 image, and it works building with tests and doc enabled.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-08-08 17:39 UTC
Newest commit scanned: 41c8b7e
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/5c312503d8/output.html

@janko-j1
Copy link
Contributor

janko-j1 commented Aug 8, 2020

Hey,
I have fixed my chroot test setup, and I detected one new issue (urwid).

I tested it against a stage3 image, and it works building with tests and doc enabled.

Did you try alot without doc?-)

@GuillaumeSeren
Copy link
Contributor Author

Did you try alot without doc?-)

:octocat:
I tested with doc only without test, but I am testing only test right now.

Closes: https://bugs.gentoo.org/732362
Package-Manager: Portage-2.3.99, Repoman-2.3.23
Signed-off-by: Guillaume Seren <[email protected]>
Package-Manager: Portage-2.3.99, Repoman-2.3.23
Signed-off-by: Guillaume Seren <[email protected]>
Package-Manager: Portage-2.3.99, Repoman-2.3.23
Signed-off-by: Guillaume Seren <[email protected]>
Closes: https://bugs.gentoo.org/736410
Package-Manager: Portage-2.3.99, Repoman-2.3.2
Signed-off-by: Guillaume Seren <[email protected]>
Closes: https://bugs.gentoo.org/736412
Package-Manager: Portage-2.3.99, Repoman-2.3.2
Signed-off-by: Guillaume Seren <[email protected]>
@GuillaumeSeren GuillaumeSeren force-pushed the mail-client/alot-add-version-bump-0.9.1 branch from 41c8b7e to 7ebb35f Compare August 8, 2020 20:15
@GuillaumeSeren
Copy link
Contributor Author

You were right @janko-j1
I fixed the sphinx dep into a BDEPEND.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-08-08 20:29 UTC
Newest commit scanned: 7ebb35f
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/ab81ae420f/output.html

Copy link
Member

@juippis juippis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to drop your urwid commits, but this is merged otherwise.

They were a bit messy and incomplete. And building urwid with [doc] doesn't work, although it doesn't work for current tree version either, could you look to fix it? Please make a new PR for urwid if you absolutely need it, but to me looks like alot can work with 2.1.

@@ -17,6 +17,10 @@ SLOT="0"
KEYWORDS="amd64 ~arm ~arm64 ~ia64 ~mips ppc ppc64 ~sparc x86 ~amd64-linux ~x86-linux"
IUSE="examples"

PATCHES=(
"${FILESDIR}/2.1-Ignore-resetting-to-invalid-locale-Closes-377.patch"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't touch the stable ebuild here at all, I'm not comfortable with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok I am creating a new urwid-2.1.0-r1 with the fix in it

IUSE="examples"

PATCHES=(
"${FILESDIR}/2.1-Ignore-resetting-to-invalid-locale-Closes-377.patch"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename the patch file better, like urwid-2.1.1-ignore-resetting-to-invalid-locale.patch.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

inherit distutils-r1

DESCRIPTION="Curses-based user interface library for Python"
HOMEPAGE="http://urwid.org/ https://pypi.org/project/urwid/"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also add the Github page here, they seem to track issues there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok


LICENSE="LGPL-2.1"
SLOT="0"
KEYWORDS="amd64 ~arm ~arm64 ~ia64 ~mips ppc ppc64 ~sparc x86 ~amd64-linux ~x86-linux"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't commit straight to stable.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@GuillaumeSeren
Copy link
Contributor Author

GuillaumeSeren commented Aug 9, 2020

And building urwid with [doc] doesn't work, although it doesn't work for current tree version either, could you look to fix it?

I did open a issue on that on gentoo and reported upstream.
https://bugs.gentoo.org/736410

Please make a new PR for urwid if you absolutely need it, but to me looks like alot can work with 2.1.

It also break alot on master now
#17059

@GuillaumeSeren
Copy link
Contributor Author

Thank you @juippis for the feedback and merge !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. maintainer-needed There is at least one affected package with no maintainer. Review it if you can.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants