[Bug 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets

[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: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets


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


tibbs@xxxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|177841                      |
              nThis|                            |




------- Additional Comments From tibbs@xxxxxxxxxxx  2007-07-04 17:22 EST -------
OK, this builds fine on x86_64 and the built package installs/uninstalls
properly on i386.  rpmlint has sprouted a new warning:

W: R-widgetTools dangerous-command-in-%postun rm
which is correct, rm is dangerous and it's used in %postun.

Honestly at this point I don't know enough about R to comment on the post
section; I'll try and get spot look over it.

This is still missing a dependency on R.

What's in %{_datadir}/R/library/%{packname}/man?  It looks like manpages.  Are
these supposed to be accessible via man?  Because the one in this package
(widgetTools.Rd) isn't.  If it's for some internal documentation browser then
that would be OK (although in either case it should be marked %doc like the rest
of the documentation).

Anyway, this is very nearly done.  Why don't you go ahead and apply for
sponsorship so that I can take care of that in anticipation of getting this
package imported.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
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]