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

--- Comment #3 from Christopher Meng <[email protected]> ---
(In reply to Matthew Farrellee from comment #2)
> (In reply to Christopher Meng from comment #1)
> > I'm interested in this package before.
> > 
> > I found many lines for el5, will you push hadoop to el5?
> > 
> > Thanks.
> 
> The target here is F20.
> 
> Are you interested in working on the dependency tree for EPEL 5?

IMO we should drop support for el5 as it's really old now in my view.

Some suggestion for the spec:

- Remove all %clean sections

- Remove BuildRoo tag

- Remove rm -rf %{buildroot} in the %install section.

- Remove %defattr(-,root,root)

And for the version tag, can we change these two from:

%global hadoop_base_version 2.0.5
%global hadoop_version %{hadoop_base_version}-alpha

to


%global hadoop_version %{version}-alpha

and put 2.0.5 into Version tag?

You can see you use nearly 20 chars to define 2.0.5, why not just put it into
Version tag?

Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=6Yggysa9Ud&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to