Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-19 Thread Panu Matilainen
Yup. This is certainly in the right direction, but what I meant is using the 
same extension for both binary and source packages (the code seems to support 
it here already), and for that the NVRS name seems strange.

-- 
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/1116#issuecomment-601099118___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-19 Thread Florian Festi
The new tags need to be added in tests/rpmgeneral.at. The change trips up a few 
more test cases:
https://semaphoreci.com/rpm-ecosystem/rpm/branches/pull-request-1116/builds/3 > 
view more > open "Fedora Latest" at very bottom > open "docker run -t rpm" > 
scroll to the very end - or run the test suite locally with `make check` (Sorry 
for the inconvenience!)


-- 
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/1116#issuecomment-601092054___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-17 Thread Peter Jones
@vathpela pushed 1 commit.

5fa11af61ae2545ad94f351379222829810bb020  Make "rpmspec -q --srpm foo.spec" say 
.src, not .%{arch}


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


Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-13 Thread Panu Matilainen
Having rpmspec default query match with what rpmbuild produces seems like a 
nice idea, the implementation is what I have some issues with, primarily that 
the new default is a magic internal behavior that you can't use it in other 
queries. I think this belongs to a tag extension, RPMTAG_PKGNEVRA or something 
like that.

Including or otherwise using stuff from build/ in lib/ is not okay, but you 
don't need access to spec struct for that, look for RPMTAG_NOSOURCE in the 
header, it should be populated at that point. And if it's not, maybe we can 
change that.



-- 
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/1116#issuecomment-598595985___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-12 Thread Peter Jones
> What about nosrc packages?

It's a bit harder to do well, as "rpmspec -q" goes through rpmcliQuery(), which 
doesn't parse the spec file, and rpmspec.c doesn't know about rpmSpec 
internals, so can't access spec->noSource without including 
rpmbuild_internals.h, but I've pushed an updated patch to do that.

-- 
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/1116#issuecomment-598228143___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Make "rpmspec -q --srpm foo.spec" say .src, not .%{arch} (#1116)

2020-03-12 Thread Michael Schroeder
What about nosrc packages?

-- 
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/1116#issuecomment-598149647___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint