[Bug 749291] Review Request: dpm-xrootd - xroot interface to the Disk Pool Manager (DPM)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--- Comment #10 from Adrien Devresse <adev88@xxxxxxxxx> ---
Conform to the requester wish, I take care of this.


First comments :

- build failure on f16/el/e6 ( I have not tested the other ) : 
  ->http://koji.fedoraproject.org/koji/taskinfo?taskID=4187213
  ->  http://koji.fedoraproject.org/koji/taskinfo?taskID=4187245

-> BuildRequires 

- %{isa}  macro is not required on BuildRequires

- BuildRequires:    xrootd-libs is implicite in xrootd-devel

- GPL 3 -> No license file or no license in headers.

- shadowutil dependency -> not needed if no user/group modifications

- It is a good pratice to use %macro or $macro but not both for lisibility

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]