Spec URL: http://vicodan.fedorapeople.org/mate-common.spec SRPM URL: http://vicodan.fedorapeople.org/mate-common-1.4.0-1.fc17.src.rpm Description: BitchX: MATE dekstop engine
oops minus the BitchX sorry, about that.. copy pasting.
I believe you're already sponsored... :) removing FE-NEEDSPONSOR, and adjusting summary to match pkg %name and %summary
Thank you Rex. Care to do the review? ;)
initial commments: no need to BuildRequires: gcc otherwise, the BuildRequires: are quite sparse, really, nothing? and, using a /usr/local prefix here is (probably) not acceptable, and try to use the default /usr prefix and %configure macro
mate-common is the first basic component of mate so all that's required is python auto make and gcc really. I can do fresh reinstall of fedora to double check but I just did that. As for configure mate uses autogen.sh and does not come with a configure script. Usr directory noted.
All 21 RPMs are built. Will upload to fedora people and polish spec files for you within 24 hours sir. Thanks for your help.
(In reply to comment #5) > mate-common is the first basic component of mate so all that's required is > python auto make and gcc really. I can do fresh reinstall of fedora to > double check but I just did that. As for configure mate uses autogen.sh and > does not come with a configure script. Usr directory noted. Dan, You can just use mock or fire scratch builds at koji now that you're a packager. They use clean build environments and help in catching build requires. http://fedoraproject.org/wiki/Using_Mock_to_test_package_builds Since MATE is a big collection of packages and would require quite a few reviews, is it worth making a tracker bug for it that would help track the reviews for its components? I'll help with the reviewing when I find free cycles. Thanks, Ankur
so, here's some suggested fixes: http://rdieter.fedorapeople.org/rpms/mate/ %changelog * Fri Jul 13 2012 Rex Dieter <rdieter> 1.4.0-2 - omit Group: tag - fix URL, Source0 - use %%configure macro - BuildArch: noarch the stuff about urls, and %configure macro usage you ought to adopt in your other mate-related pkgs too. and scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4238369
Anyone against that I help with this review ?
Hi, All RPMs, SRPMs and SPEC files have been built and are uploaded here: http://vicodan.fedorapeople.org/ Please review. This will need legal review as well. It was brought to my attention "nyan cat" is copyrighted. Thanks, Dan
(In reply to comment #7) > Since MATE is a big collection of packages and would require quite a few > reviews, is it worth making a tracker bug for it that would help track the > reviews for its components? Done. Tracker is bug #840149.
Thank you Rex and Mario. Rex, I will work with your suggestions this weekend. Mario, thanks for creating the tracker.
Hi all, I've looked around and it seems that nyan cat is indeed copyrighted and there's currently 3 copyright holders: 1) 1 copyright holder for the animated gif; 2) 1 copyright holder for the music; 3) 1 copyright holder for the video; I've taken the liberty of opening a bug report upstream requesting for this issue to be cleared out; I will talk to Stefano and Perberos about this once I grab them online. Upstream: https://github.com/perberos/Mate-Desktop-Environment/issues/104
Aditional Information: You can disable 'nyan cat' during build time if you want, you need to add this option to %configure (which you are neglecting currently): %build %configure --disable-nyancat And that should get rid of nyancat.
Dan, I've found that also another person has a few very neat packages for Fedora; You can use them to improve your specs if you want or even contact the original author, tell him about your effort and maybe you guys can do this together. Take a look at this: https://github.com/mate-desktop/fedora-packages
Nelson, Thanks for the Nyan cat comment. Regarding those packages, they are out of date and heavily patched. I have been working with Nice&Gently. He is on vacation right now. Thanks, Dan
We probably cannot distribute "nyan cat" in source format either, without permission from the copyright holder. I have not looked at the source code at all, however, if only the image of "pop tart cat" is being used here, you may wish you ask the copyright holder (prguitarman <pr>) for permission. The alternative would be to simply remove the "nyan cat" content from the source tarball entirely and make a "clean" tarball.
Thanks spot. Update spec and srpm here: spec: http://vicodan.fedorapeople.org/matespace/mate-common.spec srpm: http://vicodan.fedorapeople.org/materpms/srpms/mate/mate-common-1.4.0-3.src.rpm
updated urls; spec: http://vicodan.fedorapeople.org/matespec/mate-common.spec srpm: http://vicodan.fedorapeople.org/materpms/srpms/mate/mate-common-1.4.0-3.src.rpm
one more try for srpm link: http://vicodan.fedorapeople.org/materpms/srpms/mate-common-1.4.0-3.src.rpm Sorry.
Hi Rex, Per our conversation on IRC I have updated the SPEC and SRPM. Please review it. Once mate-common gets approved I can make mate-common a requirement for every other spec. We will also track any legality issues in the main tracker bug. Spec URL: http://vicodan.fedorapeople.org/matespec/mate-common.spec SRPM: http://vicodan.fedorapeople.org/materpms/srpms/mate-common-1.4.0-4.fc17.src.rpm Description: base files for building and installing MATE Desktop
So, I just looked closer at the gnome-common analog, http://pkgs.fedoraproject.org/gitweb/?p=gnome-common.git;a=blob;f=gnome-common.spec and see some things we could... borrow here. :) stuff like: runtime requires better summary/description so, 1. MUST: add runtime deps Requires: automake autoconf libtool gettext pkgconfig 2. SHOULD: improve pkg summary/description as suggested 3. MUST: Licensing: so, .spec says GPLv2+, and none of the included files mention licensing, except for the embedded COPYING file, which is GPLv3. can you verify with upstream (if you haven't already), their intent here? (I'm assuming some GPLv2+ (with some/all libs LGPLv2+) combo like pre-forked gnome2...) naming: ok macros: ok scriptlets: n/a sources: ok $ md5sum *.xz bc49ff6897ef2303c6464a3ca46aaf35 mate-common-1.4.0.tar.xz
Updated license. It is GPLv3+. I will do the license review on a package by package basis. Most everything is GPLv2+, GPLv3(+), lGPL as per perberos. Added gnome-common to requires field. Spec URL: http://vicodan.fedorapeople.org/matespec/mate-common.spec SRPM: http://vicodan.fedorapeople.org/materpms/srpms/mate-common-1.4.0-5.fc17.src.rpm
OK, looks good, APPROVED. curious why you added Requires: gnome-common though. ??
My mistake, misunderstood. I'll remove that. Thanks!
New Package SCM Request ======================= Package Name: mate-common Short Description: binaries for building all MATE desktop sub components Owners: vicodan rdieter Branches: f16 f17 InitialCC: vicodan rdieter
Git done (by process-git-requests).
mate-common-1.4.0-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mate-common-1.4.0-4.fc16
mate-common-1.4.0-4.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-common-1.4.0-4.fc17
Thanks Jon.
mate-common-1.4.0-5.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-common-1.4.0-5.fc17
mate-common-1.4.0-5.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mate-common-1.4.0-5.fc16
mate-common-1.4.0-5.fc17 has been pushed to the Fedora 17 stable repository.
mate-common-1.4.0-5.fc16 has been pushed to the Fedora 16 stable repository.
A little correction from me. BuildRequires: automake BuildRequires: autoconf Requires: automake Requires: autoconf Requires: libtool Requires: gettext Requires: pkgconfig because for building you need only automake and autoconf but for a working package after installation we need all the requires.
Oops indeed, missed my MUST item 1 above about adding the Requires:
Can you update the requires like in my comment 35 and release an update? Otherwise you have to add the missing Requires to every other package which need them. The sense of mate-common is this not to do for every other package.
Sure.
I give you a example: building libmatekeyring with fedora's mate-common DEBUG: + cd libmatekeyring-1.4.0 DEBUG: + /bin/chmod -Rf a+rX,u+w,g-w,o-w . DEBUG: + NOCONFIGURE=1 DEBUG: + ./autogen.sh DEBUG: /usr/bin/mate-autogen DEBUG: checking for autoconf >= 2.53... DEBUG: testing autoconf2.50... not found. DEBUG: testing autoconf... found 2.68 DEBUG: checking for automake >= 1.9... DEBUG: testing automake-1.12... not found. DEBUG: testing automake-1.11... found 1.11.1 DEBUG: checking for libtool >= 1.4.3... DEBUG: testing libtoolize... not found. DEBUG: ***Error***: You must have libtool >= 1.4.3 installed This is why Requires: libtool is missing in mate-common Pls update
mate-common-1.4.0-6.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-common-1.4.0-6.fc17
mate-common-1.4.0-6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mate-common-1.4.0-6.fc16
Done. Please test and leave karma.
yeap, it works. no need to to add BuildRequires: libtool for libmatekeyring I will leave positiv karma. thx Dan
mate-common-1.4.0-8.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-common-1.4.0-8.fc17
mate-common-1.4.0-8.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mate-common-1.4.0-8.fc16
mate-common-1.4.0-8.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.
mate-common-1.4.0-8.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.
Dan, pls give me a favor. Change Requires like in version 1.4.0-6 We need pkgconfig here, and we don't need intltool glib2-devel gtk-doc for every other mate application. Do it like this. BuildRequires: automake autoconf Requires: automake autoconf libtool gettext pkgconfig
uhm, you're requiring automake autoconf libtool and pkgconfig at runtime, and not during build? Really? Taken from here: http://vicodan.fedorapeople.org/matespec/mate-common.spec
(In reply to comment #49) > uhm, > you're requiring automake autoconf libtool and pkgconfig at runtime, and not > during build? Really? > > Taken from here: > http://vicodan.fedorapeople.org/matespec/mate-common.spec This is the sense of mate-common, adding some main requries for building mate packages. Otherwise we have to add automake autoconf libtool gettext and pkgconfig as BuildRquires in every other package.
(In reply to comment #50) > (In reply to comment #49) > > uhm, > > you're requiring automake autoconf libtool and pkgconfig at runtime, and not > > during build? Really? > > > > Taken from here: > > http://vicodan.fedorapeople.org/matespec/mate-common.spec > > This is the sense of mate-common, adding some main requries for building > mate packages. > Otherwise we have to add automake autoconf libtool gettext and pkgconfig as > BuildRquires in every other package. For running ./autogen.sh for building configure and make files of mate packages, you need this requires.
Thank you for your explanation. I absolutely understand, why requirements are necessary. The difference between BuildRequirements and Requirements is, that you need BuildRequire during build, requirements are not needed for build, but during runtime. Sadly, this didn't answer, why automake autoconf etc. are not required during build time, but for runtime.
(In reply to comment #52) > Thank you for your explanation. > > I absolutely understand, why requirements are necessary. > > The difference between BuildRequirements and Requirements is, that you need > BuildRequire during build, requirements are not needed for build, but during > runtime. > > Sadly, this didn't answer, why automake autoconf etc. are not required > during build time, but for runtime. ??? If i follow your link was given in comment 49, i see BuildRequires: automake autoconf
> ??? > If i follow your link was given in comment 49, > i see > BuildRequires: automake autoconf from the link: BuildRequires: automake autoconf Requires: automake autoconf libtool gettext pkgconfig So: why automake autoconf libtool gettext pkgconfig are required at runtime opposed to build time?
These are required at run time because mate-common provides mate-autogen which requires those at run time.
OK, I was mislead by the name of the package. It is required only for compiling other mate packages and not required at mate run-time, right? If yes, it probably should be named mate-devel, see https://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages: Fedora packages must be designed with a logical separation of files. Specifically, -devel packages must be used to contain files which are intended solely for development or needed only at build-time.
That is correct. I will have to take this up with upstream, however in its current state it makes building the dependant packages a lot simpler.
fyi, It's perfectly ok for some build-time/devel type packages to not be named -devel. There are many such examples, mate-common and gnome-common being 2 of them (others include: automake, autoconf, libtool, gcc, cmake)
(In reply to comment #58) > fyi, It's perfectly ok for some build-time/devel type packages to not be > named -devel. There are many such examples, mate-common and gnome-common > being 2 of them (others include: automake, autoconf, libtool, gcc, cmake) OK; I didn't knew that. Thank you for clarification.