* [PATCH 1/2] Revert "python-cffi: read version from notmuch version file" [not found] <20200625110045.208558-1-david@tethera.net> @ 2020-06-25 11:00 ` David Bremner 2020-06-25 11:00 ` [PATCH 2/2] bindings/python-cffi: update version from global version David Bremner 1 sibling, 0 replies; 6+ messages in thread From: David Bremner @ 2020-06-25 11:00 UTC (permalink / raw) To: notmuch This reverts commit 81057164cddf6a5c1d4c30a23767c4de8e615c1c. That version turns out not to work with some variety of pip install. --- bindings/python-cffi/setup.py | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/bindings/python-cffi/setup.py b/bindings/python-cffi/setup.py index 1effcfc6..37918e3d 100644 --- a/bindings/python-cffi/setup.py +++ b/bindings/python-cffi/setup.py @@ -1,17 +1,9 @@ -import pathlib - import setuptools -THIS_FILE = pathlib.Path(__file__).absolute() -PROJECT_ROOT = THIS_FILE.parent.parent.parent -with open(PROJECT_ROOT.joinpath('version')) as fp: - VERSION = fp.read().strip() - - setuptools.setup( name='notmuch2', - version=VERSION, + version='0.1', description='Pythonic bindings for the notmuch mail database using CFFI', author='Floris Bruynooghe', author_email='flub@devork.be', -- 2.27.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] bindings/python-cffi: update version from global version. [not found] <20200625110045.208558-1-david@tethera.net> 2020-06-25 11:00 ` [PATCH 1/2] Revert "python-cffi: read version from notmuch version file" David Bremner @ 2020-06-25 11:00 ` David Bremner 2020-06-25 13:34 ` David Bremner 1 sibling, 1 reply; 6+ messages in thread From: David Bremner @ 2020-06-25 11:00 UTC (permalink / raw) To: notmuch Copy machinery from the older python bindings --- Makefile.global | 1 + Makefile.local | 2 +- bindings/python-cffi/notmuch2/version.py | 3 +++ bindings/python-cffi/setup.py | 9 ++++++++- bindings/python/setup.py | 2 +- 5 files changed, 14 insertions(+), 3 deletions(-) create mode 100644 bindings/python-cffi/notmuch2/version.py diff --git a/Makefile.global b/Makefile.global index 0aee5876..26d9491f 100644 --- a/Makefile.global +++ b/Makefile.global @@ -47,6 +47,7 @@ SHA256_FILE=$(TAR_FILE).sha256.asc DETACHED_SIG_FILE=$(TAR_FILE).asc PV_FILE=bindings/python/notmuch/version.py +PV_FILE2=bindings/python-cffi/notmuch2/version.py # Smash together user's values with our extra values STD_CFLAGS := -std=gnu99 diff --git a/Makefile.local b/Makefile.local index 586cdf75..4b9dd0be 100644 --- a/Makefile.local +++ b/Makefile.local @@ -53,7 +53,7 @@ dist: $(TAR_FILE) update-versions: sed -i -e "s/^__VERSION__[[:blank:]]*=.*$$/__VERSION__ = \'${VERSION}\'/" \ -e "s/^SOVERSION[[:blank:]]*=.*$$/SOVERSION = \'${LIBNOTMUCH_VERSION_MAJOR}\'/" \ - ${PV_FILE} + ${PV_FILE} ${PV_FILE2} # We invoke make recursively only to force ordering of our phony # targets in the case of parallel invocation of make (-j). diff --git a/bindings/python-cffi/notmuch2/version.py b/bindings/python-cffi/notmuch2/version.py new file mode 100644 index 00000000..a5447027 --- /dev/null +++ b/bindings/python-cffi/notmuch2/version.py @@ -0,0 +1,3 @@ +# this file should be kept in sync with ../../../version +__VERSION__ = '0.30~rc2' +SOVERSION = '5' diff --git a/bindings/python-cffi/setup.py b/bindings/python-cffi/setup.py index 37918e3d..03649649 100644 --- a/bindings/python-cffi/setup.py +++ b/bindings/python-cffi/setup.py @@ -1,9 +1,16 @@ import setuptools +import os +# get the notmuch version number without importing the notmuch module +version_file = os.path.join(os.path.dirname(__file__), + 'notmuch2', 'version.py') +exec(compile(open(version_file).read(), version_file, 'exec')) +assert '__VERSION__' in globals(), \ + 'Failed to read the notmuch binding version number' setuptools.setup( name='notmuch2', - version='0.1', + version=__VERSION__, description='Pythonic bindings for the notmuch mail database using CFFI', author='Floris Bruynooghe', author_email='flub@devork.be', diff --git a/bindings/python/setup.py b/bindings/python/setup.py index d986f0c6..d4c5c844 100644 --- a/bindings/python/setup.py +++ b/bindings/python/setup.py @@ -24,7 +24,7 @@ from distutils.core import setup # get the notmuch version number without importing the notmuch module version_file = os.path.join(os.path.dirname(__file__), - 'notmuch', 'version.py') + 'notmuch2', 'version.py') exec(compile(open(version_file).read(), version_file, 'exec')) assert '__VERSION__' in globals(), \ 'Failed to read the notmuch binding version number' -- 2.27.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] bindings/python-cffi: update version from global version. 2020-06-25 11:00 ` [PATCH 2/2] bindings/python-cffi: update version from global version David Bremner @ 2020-06-25 13:34 ` David Bremner 2020-06-29 21:02 ` Floris Bruynooghe 0 siblings, 1 reply; 6+ messages in thread From: David Bremner @ 2020-06-25 13:34 UTC (permalink / raw) To: notmuch David Bremner <david@tethera.net> writes: > Copy machinery from the older python bindings > > +# get the notmuch version number without importing the notmuch module > +version_file = os.path.join(os.path.dirname(__file__), > + 'notmuch2', 'version.py') > +exec(compile(open(version_file).read(), version_file, 'exec')) > +assert '__VERSION__' in globals(), \ > + 'Failed to read the notmuch binding version number' I wrote a cover letter for this, but that seems to have gotten lost. My main point was I'm not sure why this is better than Floris's version, since they both read a file when setup.py is run. I don't understand (or use) pip, so someone else will have to figure this out. If the constraint is that the version has to be hardcoded in setup.py then (as much as that sounds like a design mistake), we can apply similar sed hackery directly to setup.py. Perhaps someone can remember why we didn't do that for the old python bindings. d ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] bindings/python-cffi: update version from global version. 2020-06-25 13:34 ` David Bremner @ 2020-06-29 21:02 ` Floris Bruynooghe 2020-06-30 0:22 ` [PATCH] bindings/python-cffi: copy version file into bindings dir David Bremner 0 siblings, 1 reply; 6+ messages in thread From: Floris Bruynooghe @ 2020-06-29 21:02 UTC (permalink / raw) To: David Bremner, notmuch On Thu 25 Jun 2020 at 10:34 -0300, David Bremner wrote: > David Bremner <david@tethera.net> writes: > >> Copy machinery from the older python bindings > >> >> +# get the notmuch version number without importing the notmuch module >> +version_file = os.path.join(os.path.dirname(__file__), >> + 'notmuch2', 'version.py') >> +exec(compile(open(version_file).read(), version_file, 'exec')) >> +assert '__VERSION__' in globals(), \ >> + 'Failed to read the notmuch binding version number' > > I wrote a cover letter for this, but that seems to have gotten lost. My > main point was I'm not sure why this is better than Floris's version, > since they both read a file when setup.py is run. I don't understand (or > use) pip, so someone else will have to figure this out. If the > constraint is that the version has to be hardcoded in setup.py then (as > much as that sounds like a design mistake), we can apply similar sed > hackery directly to setup.py. Perhaps someone can remember why we didn't > do that for the old python bindings. For some reason this is the only mail in this thread I have, so I don't actually know the patch. I think it can be simpler though, is it possible to copy the toplevel version file into bindings/python-cffi/version in the part of the build that would otherwise do the sed magic? Then setup.py only needs to look for the version file in the same directory as itself instead of finding the toplevel of the repo. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] bindings/python-cffi: copy version file into bindings dir 2020-06-29 21:02 ` Floris Bruynooghe @ 2020-06-30 0:22 ` David Bremner 2020-07-03 11:37 ` David Bremner 0 siblings, 1 reply; 6+ messages in thread From: David Bremner @ 2020-06-30 0:22 UTC (permalink / raw) To: Floris Bruynooghe, David Bremner, notmuch Attempt to avoid breaking "pip install ." As far as I can tell, we need to have a copy (not just a relative symlink) of the version file. --- Makefile.local | 1 + bindings/python-cffi/setup.py | 8 +------- bindings/python-cffi/version | 1 + 3 files changed, 3 insertions(+), 7 deletions(-) create mode 100644 bindings/python-cffi/version diff --git a/Makefile.local b/Makefile.local index 586cdf75..314238ef 100644 --- a/Makefile.local +++ b/Makefile.local @@ -54,6 +54,7 @@ update-versions: sed -i -e "s/^__VERSION__[[:blank:]]*=.*$$/__VERSION__ = \'${VERSION}\'/" \ -e "s/^SOVERSION[[:blank:]]*=.*$$/SOVERSION = \'${LIBNOTMUCH_VERSION_MAJOR}\'/" \ ${PV_FILE} + cp version bindings/python-cffi # We invoke make recursively only to force ordering of our phony # targets in the case of parallel invocation of make (-j). diff --git a/bindings/python-cffi/setup.py b/bindings/python-cffi/setup.py index 1effcfc6..b0060835 100644 --- a/bindings/python-cffi/setup.py +++ b/bindings/python-cffi/setup.py @@ -1,14 +1,8 @@ -import pathlib - import setuptools - -THIS_FILE = pathlib.Path(__file__).absolute() -PROJECT_ROOT = THIS_FILE.parent.parent.parent -with open(PROJECT_ROOT.joinpath('version')) as fp: +with open('version') as fp: VERSION = fp.read().strip() - setuptools.setup( name='notmuch2', version=VERSION, diff --git a/bindings/python-cffi/version b/bindings/python-cffi/version new file mode 100644 index 00000000..71971d9b --- /dev/null +++ b/bindings/python-cffi/version @@ -0,0 +1 @@ +0.30~rc2 -- 2.27.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] bindings/python-cffi: copy version file into bindings dir 2020-06-30 0:22 ` [PATCH] bindings/python-cffi: copy version file into bindings dir David Bremner @ 2020-07-03 11:37 ` David Bremner 0 siblings, 0 replies; 6+ messages in thread From: David Bremner @ 2020-07-03 11:37 UTC (permalink / raw) To: Floris Bruynooghe, notmuch David Bremner <david@tethera.net> writes: > Attempt to avoid breaking "pip install ." > > As far as I can tell, we need to have a copy (not just a relative > symlink) of the version file. Applied, as part of 0.30~rc3. Let me know if this is still broken for pip users. I tried "pip install .", but that's all I know. d _______________________________________________ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send an email to notmuch-leave@notmuchmail.org ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-07-03 11:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20200625110045.208558-1-david@tethera.net>
2020-06-25 11:00 ` [PATCH 1/2] Revert "python-cffi: read version from notmuch version file" David Bremner
2020-06-25 11:00 ` [PATCH 2/2] bindings/python-cffi: update version from global version David Bremner
2020-06-25 13:34 ` David Bremner
2020-06-29 21:02 ` Floris Bruynooghe
2020-06-30 0:22 ` [PATCH] bindings/python-cffi: copy version file into bindings dir David Bremner
2020-07-03 11:37 ` David Bremner
Code repositories for project(s) associated with this public inbox https://yhetil.org/notmuch.git/ This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).