Skip to content

Conversation

@lmr
Copy link
Member

@lmr lmr commented Feb 28, 2023

I forgot to make that update when cutting 1.7.0.

@lmr
Copy link
Member Author

lmr commented Feb 28, 2023

@clebergnu when you have some time, please update the Fedora packages. I have been doing some package maintenance for Amazon Linux, so I can also help with maintenance in case you want.

@lmr lmr requested review from clebergnu and ldoktor February 28, 2023 15:18
Copy link
Contributor

@ldoktor ldoktor left a comment

Choose a reason for hiding this comment

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

Hello Lucas, there is one more file to be changed:

diff --git a/Makefile b/Makefile
index 6c2dc73..2fe0ea5 100644
--- a/Makefile
+++ b/Makefile
@@ -3,7 +3,7 @@ PYTHON_DEVELOP_ARGS=$(shell if ($(PYTHON) setup.py develop --help 2>/dev/null |
 DESTDIR=/
 BUILDIR=$(CURDIR)/debian/aexpect
 PROJECT=aexpect
-VERSION="1.6.4"
+VERSION="1.7.0"
 COMMIT=$(shell git log --pretty=format:'%H' -n 1)
 SHORT_COMMIT=$(shell git log --pretty=format:'%h' -n 1)
 COMMIT_DATE=$(shell git log --pretty='format:%cd' --date='format:%Y%m%d' -n 1)

I forgot to make that update when cutting 1.7.0.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@amazon.com>
@lmr lmr force-pushed the fix/rpm-build-spec-file branch from 6ba39e6 to 6184ddb Compare March 1, 2023 16:07
@lmr
Copy link
Member Author

lmr commented Mar 1, 2023

@ldoktor good point, updated.

Copy link
Contributor

@ldoktor ldoktor left a comment

Choose a reason for hiding this comment

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

Thank you, seems to work well although I'm wondering how the make rpm-release will work (maybe a new tag will be required, but IIRC Fedora uses different targets so perhaps it's not necessary; let's wait for @clebergnu or @mmathesius)

@ldoktor ldoktor merged commit 5182d37 into main Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants