Spec URL: http://etoys.laptop.org/srpm/squeak-vm-3.9-11olpc2.spec SRPM URL: http://etoys.laptop.org/srpm/squeak-vm-3.9-11olpc2.src.rpm Description: Squeak VM for OLPC Etoys activity
Some quick comments about your SPEC file. you don't cleaned the buildroot in the %clean section and on the beginning of the %install section.
Some additional comments: - First in the %install section the buildroot was cleaned, but no in the %clean sectioon. Sorry for my mistake on comment #1 - you don't used the %{?_smp_mflags} makro in the make step.
Fixed clean and _smp_mflags Spec URL: http://etoys.laptop.org/srpm/squeak-vm-3.9-11olpc3.spec SRPM URL: http://etoys.laptop.org/srpm/squeak-vm-3.9-11olpc3.src.rpm
Created attachment 159280 [details] Mock build log
Good: + Package meets naming guidelines + SPEC filename match with package base name + License is MIT + License tag matches with license included in upstream tar ball. + SPEC file is written in English + Package has correct buildroot + Package has not redundant BuildRequires + File list of package doesn't contains duplicates entries + File list contains no files or directories own by other packages Bad. - Don't use Vendor tag - Source tag contains not a full qualified URI - Unnecessary Provide tag - Condition before deleting of the build root is not require - Missing Version entries in the changelog entries - Inproper use of the rpm macros in the %file section - Package doesn't contains a %defattr statemend - Version 3.9-11 seems not to be an official stable version - Package doesn't contains a %doc section - Package doens't contains verbatim copy of the license text, but you can find the license text in the upstream tar ball - Build doesn't use compiler flags in $RPM_OPT_FLAGS - Unnecessary Prefix tag - If you wnat to include all files and directories belang a specific directory, the entry in the file list must ands with a slash - Your package contains file which should be belongs th the nonexisting %doc section - Rpmlint complaints on source rpm: rpmlint squeak-vm-3.9-11olpc3.src.rpm W: squeak-vm hardcoded-prefix-tag %{prefix} W: squeak-vm unversioned-explicit-provides %{name}-%{version} W: squeak-vm setup-not-quiet W: squeak-vm rpm-buildroot-usage %build make ROOT=%{buildroot} %{?_smp_mflags} W: squeak-vm mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9) - Rpmlint complaints on binary rpm: rpmlint squeak-vm-3.9-11olpc3.x86_64.rpm E: squeak-vm binary-or-shlib-defines-rpath /usr/lib/squeak/3.9-11/squeak ['/usr/lib'] - Mock build failed on Devel (ppc64). Build log will be attached Special question: Do you search a sponsor. If yes, Unfortunately, I can't sponsor you. because I'm not a sponsor.
Thanks Jochen, I'll work on the "bad" list. The build failure is due to a missing libXt-devel BuildRequire. And I've got someone willing to sponsor me but he's on vacation atm.
Ping Bert.
Heh, thanks for the ping. Now *I* am on vacation, should get back to this at the end of August.
Bert, you are not yet sponsored according to the Fedora Account System, please read: http://fedoraproject.org/wiki/PackageMaintainers/Join http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored In case there OLPC contributors do not need to be sponsored, please tell me.
Bert says in an e-mail: This doesn't work because I cannot sign the CLA on behalf of Viewpoints. I'm only a consultant for them, not a legal representative. I have explained the reasoning for the CLA but have also heard whole companies can sign a CLA which may alleviate Bert's burden. Kicking to FE-Legal so that they can explain more and give Bert options. This is also an issue with bug #247984
What's the status of this Review Request? Anther one question - why this package mentions OLPC in its name? Looks like this package is a general purpose Smaltalk VM and nor contains special OLPC code neither specially designed for OLPC.
Status is same as #247984. This package is compiled for OLPC, for example, the OpenGL support was taken out because OLPC does not ship libGL, and special Sugar-attribute support was added that is only useful when actually running under Sugar. But you are right in so far as this VM should work on regular machines, too.
please advise as to where the source code for this was originally obtained. ftp.squeak.org does not contain the same source tarball as is bundled in this package. Also note, significant portions of this package appear to be dual licensed under the LGPLv2.1+ and the Squeak license, neither of which are MIT. ./platforms/Cross/plugins/JPEGReadWriter2Plugin/jcomapi.c notes it is licensed and to read a README file, but no such file exists (contrary to the Independent JPEG Group's license). It might be considered free enough, but that's not entirely clear. There's a boatload of MP3 plugin code here. MP3 is patented. ./platforms/Cross/plugins/SoundCodecPrims/ is missing a COPYRIGHT file containing the license for code in that dir too. There's a lot of stuff under a Sun copyright with a permissive license. There's stuff under the Squeak license, not dual-licensed. There's Perl-Compatible-Regular-Expressions which is yet another license... There's GPLv2+ code in ./platforms/unix/plugins/VideoForLinuxPlugin/ccvt_types.h Debian won't carry this even in non-free. This scares me. Thanks to LaserJock for pointing out the legal mess that this code appears to be. Contining to block FE-Legal.
Yeaaah. I'm closing this one as CANTFIX. It will need a LOT of work to be made clean for Fedora, not to mention the MP3 problems and the Squeak license (non-free).
In your case I will suggest to open the review on rpmfusion.org because they have no issues with this kind of licensing issue.