[Bug 427275] Review Request: python-tgexpandingformwidget - A repeating form widget for TurboGears

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request:  python-tgexpandingformwidget - A repeating form widget for TurboGears


https://bugzilla.redhat.com/show_bug.cgi?id=427275


tibbs@xxxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




------- Additional Comments From tibbs@xxxxxxxxxxx  2008-01-17 17:00 EST -------
I think you can rremove the pyver macro from the top of the spec, as it seems to
be unused.

Shouldn't this have a dependency on TurboGears in some fashion?  rpmbuild
doesn't really generate proper dependencies for python code, so you have to do
it manually.  On the other hand, maybe it doesn't need TG; I'm honestly not an
expert, so I'll leave it to you.

Please use either %{buildroot} or $RPM_BUILD_ROOT but not both in the same specfile.

That's an easy fix, so I'll approve this package and you can fix it when you
check in.

Checklist:
* source files match upstream:
   5faa544f0ec0ea914b4b4719b3a8f54a1be732bf302b7ae918715f460c621b63  
   TGExpandingFormWidget-0.1.3.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named and is cleanly written
X specfile does not use macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint is silent.
? final provides and requires (maybe needs turbogears)
   python-tgexpandingformwidget = 0.1.3-4.fc9
  =
   python(abi) = 2.5

* %check is not present; no test suite upstream.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

APPROVED; just s/\$RPM_BUILD_ROOT/%{buildroot}/ when you check in.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]