Bug 465159
Summary: | Review Request: tcexam - A Web-based Computer-Based Assessment (CBA) software | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Oron Peled <oron> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | aquini, fedora-package-review, notting |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-08-07 15:01:05 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 201449 |
Description
Oron Peled
2008-10-01 20:46:11 UTC
Spec URL: http://oron.fedorapeople.org/tcexam/tcexam.spec SRPM URL: http://oron.fedorapeople.org/tcexam/tcexam-7.0.006-1.fc10.src.rpm * Bumped to upstream version 7.0.006 * Better configuration via included tcexam-setdef script (used both during build and in post-install). * Included Hebrew support patch + translation (sent and accepted by upstream, patch should be removed on next release). Wow, this is ancient. Are you still interested in contributing this package? If so, here are a few comments: I note that the current version has a significantly larger version number, which is not surprising. The selinux subpackage is going to be scary for most reviewers; I certainly don't feel qualified to review it. Have the selinux folks (dwalsh, etc.) had a look at your policy? I have significant doubts about files in /usr/share being configuration files. Group is completely unimportant; put whatever you want there. The proper way to classify such applications is using comps. If tcpdf.org is actually a separate package (separate tarball, differing release cycle), it cannot be bundled as part of this one. The fonts cannot be bundled; they must be packaged (if they aren't already) and the code must use the packaged versions. Don't try to output information in scriptlets; you can't expect that it will be seen by anyone. > Wow, this is ancient. Are you still interested in contributing this package? Yes, I am. I rebuilt an updated package, NOT to be pushed to Fedora as is, but to be used as a working reference while we work on resolving the different issues. It is located in: SPEC: http://oron.fedorapeople.org/tcexam/tcexam.spec SRPM: http://oron.fedorapeople.org/tcexam/tcexam-9.0.035-1.fc11.src.rpm Work plan: * Package tcpdf.org: - Font issues -- started with Fedora font SIG: https://www.redhat.com/archives/fedora-fonts-list/2009-November/msg00013.html - Maybe need to package ttf2ufm and/or afm2pfm - Coordinate with moodle packager. During my search for ttf2ufm upstream I found that tcpdf and its fonts are bundled with moodle (including in the Fedora package :-O * Verify selinux policy with dwalsh * Finally package tcexam.org itself. Can you review all this stuff, step by step, as I move along? - If yes, I will start working on the items. For every new package needed, I will add here a link to its bug report so you can easily find them. - If you do not have the time for this, I will regretfully have to return to my numb mode until someone find this bug report... Thanks, I'm willing to answer questions if you ask and I actually understand enough to answer, but I'm not sure there's any point in anyone reviewing this package until the known issues are closer to being fixed. The moodle problem is indeed an issue; it might have been missed on review, or it might have been added after review. Regardless, tcpdf should certainly be split out so that both packages can use it. PING It's been almost ten months with no progress; This bug should be closed soon if there is no response, shouldn't it? Due to the lack of response, this review is now considered as stalled. I'm closing this bug just as described in Fedora's Policy for stalled package reviews http://fedoraproject.org/wiki/Policy_for_stalled_package_reviews That's OK on my part. If/when I have more time, I'll try to refactor tcpdf which is the real blocker here. Until than please note that moodle (which is packaged in Fedora) carries a private copy of tcpdf just like tcexam -- which is definitely contra Fedora policies... :-( |