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

rpm: update to 4.16.0 #26642

Closed
wants to merge 2 commits into from
Closed

rpm: update to 4.16.0 #26642

wants to merge 2 commits into from

Conversation

glaulher
Copy link
Contributor

No description provided.

@@ -29,3 +29,4 @@ index 992d9ac..5f43b57 100644

#include <rpm/rpmlog.h>
#include "rpmio/digest.h"

Copy link
Member

Choose a reason for hiding this comment

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

Remove this newline

python_version=2 #unverified
distfiles="http://ftp.rpm.org/releases/rpm-${version%.*}.x/rpm-${version}.tar.bz2"
checksum=ca5974e9da2939afb422598818ef187385061889ba766166c4a3829c5ef8d411
python_version=2
Copy link
Member

Choose a reason for hiding this comment

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

Is 2 correct? It's linked against python3...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

deleting this line, the package does not compile.

Copy link
Member

Choose a reason for hiding this comment

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

No, I think you should be using python_version=3

Copy link
Member

Choose a reason for hiding this comment

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

ping here

Copy link
Member

Choose a reason for hiding this comment

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

It seems that there is exactly one Python script installed by rpm, and it works with Python 3 (I can at least get the help text to display). Please change to python_version=3 as @ericonr recommends.

@ericonr
Copy link
Member

ericonr commented Nov 23, 2020

The patch you made is patching the wrong file, you need to apply

diff --git a/srcpkgs/rpm/patches/fix-musl-rpmdb.patch b/srcpkgs/rpm/patches/fix-musl-rpmdb.patch
index e1e48b3b0d..2c0317ee5a 100644
--- a/srcpkgs/rpm/patches/fix-musl-rpmdb.patch
+++ b/srcpkgs/rpm/patches/fix-musl-rpmdb.patch
@@ -1,12 +1,11 @@
---- rpmdb.c
-+++ rpmdb.c
-@@ -1,7 +1,9 @@
-#include "system.h"
-
+--- lib/rpmdb.c
++++ lib/rpmdb.c
+@@ -8,6 +8,8 @@
+ #include <utime.h>
+ #include <errno.h>
+ #include <dirent.h>
++#include <sys/stat.h>
 +#include <fcntl.h>
-+#include <sys / stat.h>
-#include <popt.h>
-#include <rpm/rpmcli.h>
-#include <rpm/rpmdb.h>
-#include "cliutils.h"
-#include "debug.h"
+ 
+ #ifndef	DYING	/* XXX already in "system.h" */
+ #include <fnmatch.h>

to your repository instead.

The final patch should look like

--- lib/rpmdb.c
+++ lib/rpmdb.c
@@ -8,6 +8,8 @@
 #include <utime.h>
 #include <errno.h>
 #include <dirent.h>
+#include <sys/stat.h>
+#include <fcntl.h>
 
 #ifndef	DYING	/* XXX already in "system.h" */
 #include <fnmatch.h>

Also, rebase your branch so the merge commit goes away.

@glaulher
Copy link
Contributor Author

the package compiled in musl but took 2h6m46s

@ericonr
Copy link
Member

ericonr commented Nov 23, 2020

Your local repository might have been out of date, which would have made it build the outdated dependencies. Do git remote add upstream https://github.com/void-linux/void-packages.git then git pull upstream master --autostash

@glaulher
Copy link
Contributor Author

for some reason trying to rebase I pushed the push up incorrectly, how do I fix it?

@ericonr
Copy link
Member

ericonr commented Nov 23, 2020

git rebase upstream/master should do it

@ahesford
Copy link
Member

Deeper and deeper!

Try this from the root of your clone, while you are in the rpm branch:

git fetch --all --prune
cp srcpkgs/rpm/{template,patches/fix-musl-rpmdb.patch} .
git reset --hard upstream/master
mv fix-musl-rpmdb.patch srcpkgs/rpm/patches/
mv template srcpkgs/rpm/
git add srcpkgs/rpm
git commit -m 'rpm: update to 4.16.0.'
git push -f

@glaulher
Copy link
Contributor Author

Cada vez mais fundo!

Tente fazer isso a partir da raiz do seu clone, enquanto você está no rpmbranch:

git fetch --all --prune
cp srcpkgs/rpm/{template,patches/fix-musl-rpmdb.patch} .
git reset --hard upstream/master
mv fix-musl-rpmdb.patch srcpkgs/rpm/patches/
mv template srcpkgs/rpm/
git add srcpkgs/rpm
git commit -m 'rpm: update to 4.16.0.'
git push -f

thanks for your help and ericonr's help,
I think it's ok now.

version=4.15.1
revision=4
version=4.16.0
revision=2
Copy link
Member

Choose a reason for hiding this comment

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

This should be revision=1 when the version is new.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the version has been fixed, thanks.

@ericonr
Copy link
Member

ericonr commented Dec 17, 2020

Você ainda tinha dois commits (pode ver aqui https://github.com/void-linux/void-packages/pull/26642/commits ). Eu juntei os dois e aproveitei para atualizar para 4.16.1.2.

@glaulher
Copy link
Contributor Author

thank you.

@glaulher glaulher deleted the rpm branch January 1, 2021 15:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants