Bug 226525

Summary:	Merge Review: velocity
Product:	[Fedora] Fedora	Reporter:	Nobody's working on this, feel free to take it <nobody>
Component:	Package Review	Assignee:	Alexander Kurtakov <akurtako>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	akurtako, sochotni, viveklak
Target Milestone:	---	Flags:	akurtako: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-11-03 17:53:07 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:

Description Nobody's working on this, feel free to take it 2007-01-31 21:15:00 UTC

Fedora Merge Review: velocity

http://cvs.fedora.redhat.com/viewcvs/devel/velocity/
Initial Owner: vivekl

Comment 1 Alexander Kurtakov 2010-10-14 12:20:06 UTC

I'll do this one.

Comment 2 Alexander Kurtakov 2010-10-14 12:45:23 UTC

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 17:38:42 UTC

All comments should be fixed now:

https://koji.fedoraproject.org/koji/taskinfo?taskID=2574591

Comment 4 Alexander Kurtakov 2010-11-03 17:52:41 UTC

Thanks,

APPROVED.

Comment 5 Alexander Kurtakov 2010-11-03 17:53:07 UTC

As this is merge review - closing.