Spec URL: http://ispbrasil.com.br/ytnef/ytnef.spec SRPM URL: http://ispbrasil.com.br/ytnef/ytnef-2.6-2.src.rpm Description: Yerase's TNEF Stream Reader. Can take a TNEF Stream (winmail.dat) sent from Microsoft Outlook (or similar products) and extract the attachments, including construction of Contact Cards & Calendar entries.
koji scratch build in dist-f11 http://koji.fedoraproject.org/koji/taskinfo?taskID=1124686
Note that correct SRPM URL should be: http://ispbrasil.com.br/ytnef/ytnef-2.6-2.fc10.src.rpm At initial glance, can you change the license to GPLv2+?. Also, it's a little cleaner to use the %{__make}, %{__rm} macros if possible, and you might consider including ChangeLog as part of the documentation. I'll do a formal review tomorrow. Going to bed right now. :-)
I am not clear about the license, I belive the correct license should be the same as already included libytnef-devel, GPL+ please l@@k new spec. http://ispbrasil.com.br/ytnef/ytnef.spec http://ispbrasil.com.br/ytnef/ytnef-2.6-3.fc11.src.rpm koji scratch build (dist-f11) ------------ http://koji.fedoraproject.org/koji/taskinfo?taskID=1256395
Sorry for the very long delay on this. :-( Going through the review I realized I can't even install this on Fedora 10: ytnef-2.6-3.fc10.i386 from ../RPMS/ytnef-2.6-3.fc10.i386.rpm has depsolving problems --> Missing Dependency: perl(Mail::Mailer::test) is needed by package ytnef-2.6-3.fc10.i386 (../RPMS/ytnef-2.6-3.fc10.i386.rpm) Error: Missing Dependency: perl(Mail::Mailer::test) is needed by package ytnef-2.6-3.fc10.i386 (../RPMS/ytnef-2.6-3.fc10.i386.rpm) (yum localinstall results). Maybe this package is provided by rawhide? I'll check. a yum provides on my F10 repos comes up with nothing however.
(In reply to comment #4) please hold, I will check this soon.
Any new information on this?
look again http://ispbrasil.com.br/ytnef/ytnef.spec http://ispbrasil.com.br/ytnef/ytnef-2.6-4.fc11.src.rpm --- http://koji.fedoraproject.org/koji/taskinfo?taskID=1380867
ping
My informal review: - Package build fine on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1612465 dist-f11 - Silent rpmlint output I think is all done with this package and should be approved :)
Sorry guys, got super busy with work and have seriously neglected Fedora lately. I will do the formal review today.
rpmlint output: ytnef.i586: I: checking1 packages and 0 specfiles checked; 0 errors, 0 warnings. MUST: The License field in the package spec file must match the actual license. -- Looks like the license string for this should be set to GPLv2+ not GPL+ All other MUST items check out. All SHOULD items check out. ACCEPTED
New Package CVS Request ======================= Package Name: ytnef Short Description: Yerase's TNEF Stream Reader Owners: itamarjp Branches: F-10 F-11 InitialCC:
CVS done.
ytnef-2.6-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/ytnef-2.6-4.fc11
ytnef-2.6-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/ytnef-2.6-4.fc10
ytnef-2.6-4.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ytnef'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-8853
ytnef-2.6-4.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ytnef'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-8871
ytnef-2.6-4.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
ytnef-2.6-4.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.