Bug 226525 - Merge Review: velocity
Merge Review: velocity
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Alexander Kurtakov
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 16:15 EST by Nobody's working on this, feel free to take it
Modified: 2010-11-03 13:53 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-11-03 13:53:07 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
akurtako: fedora‑review+


Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 16:15:00 EST
Fedora Merge Review: velocity

http://cvs.fedora.redhat.com/viewcvs/devel/velocity/
Initial Owner: vivekl@redhat.com
Comment 1 Alexander Kurtakov 2010-10-14 08:20:06 EDT
I'll do this one.
Comment 2 Alexander Kurtakov 2010-10-14 08:45:23 EDT
Issues:
1. newer version is available
2. File permissions are not using defaults.
3. manual, javadoc and demo subpackages are not installing License
4. included jars are not removed in prep
5. %{_javadocdir}/%{name} is not created/installed
6. both $RPM_BUILD_ROOT and %{buildroot} are used
Comment 3 Stanislav Ochotnicky 2010-11-03 13:38:42 EDT
All comments should be fixed now:

https://koji.fedoraproject.org/koji/taskinfo?taskID=2574591
Comment 4 Alexander Kurtakov 2010-11-03 13:52:41 EDT
Thanks,

APPROVED.
Comment 5 Alexander Kurtakov 2010-11-03 13:53:07 EDT
As this is merge review - closing.

Note You need to log in before you can comment on or make changes to this bug.