Re: [Rpm-maint] [rpm-software-management/rpm] %attr license files by default (#1090)

2020-03-30 Thread Panu Matilainen
Should be done as a part of #567.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1090#issuecomment-606054657___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] find-debuginfo.sh: cpio looking into wrong path (#322)

2020-03-30 Thread Panu Matilainen
Ok, still easily reproducable by commenting out this:
https://src.fedoraproject.org/rpms/timeshift/blob/master/f/timeshift.spec#_71

Whether there's anything rpm can actually do about this I dunno.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/322#issuecomment-605993643___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Flush 1998 vintage dirent.h and fcntl.h compatibility mess from system.h (#1146)

2020-03-30 Thread soig
Looks good

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1146#issuecomment-605953855___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Add an ArchiveX header (#372)

2020-03-30 Thread Florian Festi
This is a valid use case. But I dislike the idea of adding a new tag for this. 
I'd rather like to see something that is closer to the hack without its 
drawbacks
`Source1: https://foo.bar/baz -> ba-1.2.tgz` or
`Source1: https://foo.bar/baz : ba-1.2.tgz`


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/372#issuecomment-605953186___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)

2020-03-30 Thread Panu Matilainen
Right, that's a fair point.

Ultimately there should indeed be just one way to build the bindings, 
maintaining multiple ways was never a good idea anyway. OTOH calling setup.py 
from inside automake sounds like a nightmare too.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605952790___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)

2020-03-30 Thread ニール・ゴンパ
Without the setup.py, we don't have the egg-info data which other projects use 
to depend on the Python bindings. The Autofoo should probably just be reworked 
to call setup.py instead.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605949977___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: macro for checking endianess (#365)

2020-03-30 Thread Florian Festi
Needs to be looked at when re-doing the architecture handling.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/365#issuecomment-605946854___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)

2020-03-30 Thread Panu Matilainen
The issue here is pretty convoluted: the setup.py method of building was 
originally added primarily to support shipping python-bindings as a separate 
tarball, to support building multiple versions of the bindings. The former 
never happened, and to support Py 2 vs 3 people have been building the bindings 
in-tree with the setup.py method, which never really really worked. 

Now that that Python 2 is no longer supported at all, I think the solution to 
this is either axe the setup.py method entirely, or make it error-out if trying 
to build in-tree.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605944390___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: Decouple build, host, and target platforms to enable cross-compiling RPM and RPMs (#103)

2020-03-30 Thread Panu Matilainen
This needs to be reviewed in the context of the architecture redesign whenever 
that happens.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/103#issuecomment-605925518___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: read sources checksums from the SPEC file and verify them (#463)

2020-03-30 Thread Nils Philippsen
Unfortunately, the minimalistic solution is used for something else already: 
URL fragments are used to have a different file name from what a download from 
the URL would normally yield, e.g. if an on-the-fly generated upstream archive 
doesn't contain the package name.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/463#issuecomment-605923682___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Flush 1998 vintage dirent.h and fcntl.h compatibility mess from system.h (#1146)

2020-03-30 Thread Panu Matilainen
Added similar cleanup for fcntl.h, and rebased to fix conflict from the 
vfs-cleanup.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1146#issuecomment-605877061___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Flush 1998 vintage dirent.h compatibility mess from system.h (#1146)

2020-03-30 Thread Panu Matilainen
@pmatilai pushed 1 commit.

6ac1880a100c8ab4cf532f5959bf84fe2af9dfdb  Flush 1998 vintage 
fcntl-compatibility mess from system.h


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1146/files/e6d89e45f910210b0397ed9c6a848eafe17c78c8..6ac1880a100c8ab4cf532f5959bf84fe2af9dfdb
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Flush 1998 vintage dirent.h compatibility mess from system.h (#1146)

2020-03-30 Thread Panu Matilainen
Added a missed dirent.h in fts.c (now there's another pile of ... to clean up)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1146#issuecomment-605866634___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] Flush 1998 vintage dirent.h compatibility mess from system.h (#1146)

2020-03-30 Thread Panu Matilainen
dirent.h and struct dirent are actually standard on this millenium, the
only thing that isnt is d_type member for which we have and retain
a specific test in configure.ac. Include dirent.h where needed,
relatively few places do which makes it even a bigger insult to have
this included from system.h.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/1146

-- Commit Summary --

  * Flush 1998 vintage dirent.h compatibility mess from system.h

-- File Changes --

M lib/backend/ndb/rpmpkg.c (1)
M lib/backend/ndb/rpmxdb.c (1)
M lib/rpmdb.c (1)
M rpmio/rpmglob.c (5)
M rpmio/rpmio.c (1)
M system.h (17)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/1146.patch
https://github.com/rpm-software-management/rpm/pull/1146.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1146
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Flush 20+ year old statfs() jungle, always use standard statvfs() (#1143)

2020-03-30 Thread Florian Festi
Merged #1143 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1143#event-3177469691___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Fix phrasing & package one more man page (#1145)

2020-03-30 Thread Florian Festi
Thanks for the patches and having a close look at the alpha!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1145#issuecomment-605848969___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Fix phrasing & package one more man page (#1145)

2020-03-30 Thread Florian Festi
Merged #1145 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1145#event-3177397915___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] Fix phrasing & package one more man page (#1145)

2020-03-30 Thread soig

You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/1145

-- Commit Summary --

  * fix phrasing
  * also package rpm-plugins.8

-- File Changes --

M doc/Makefile.am (1)
M doc/rpm-plugin-prioreset.8 (2)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/1145.patch
https://github.com/rpm-software-management/rpm/pull/1145.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1145
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] actually include the new man pages (#1144)

2020-03-30 Thread Panu Matilainen
@pmatilai approved this pull request.

Doh! Thanks :slightly_smiling_face: 



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1144#pullrequestreview-383606226___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] actually include the new man pages (#1144)

2020-03-30 Thread Panu Matilainen
Merged #1144 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1144#event-3177329018___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] actually include the new man pages (#1144)

2020-03-30 Thread soig
https://rpm.org/wiki/Releases/4.16.0 says Add man pages for all plugins 
and rpm2archive (#1016)
However RPM 4.16.0 alpha doesnt actually include them…
Ahem

This pull request actually includes the new man pages  the tarball and thus the 
vendor rpms… :-)
Thanks
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/1144

-- Commit Summary --

  * actually include the new man pages

-- File Changes --

M doc/Makefile.am (5)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/1144.patch
https://github.com/rpm-software-management/rpm/pull/1144.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1144
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint