[Bug 783655] Review Request: aimage - Advanced Disk Imager

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Volker Fröhlich <volker27@xxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |volker27@xxxxxx

--- Comment #1 from Volker Fröhlich <volker27@xxxxxx> 2012-01-21 18:51:28 EST ---
You can remove defattr.

It seems to me, there is no afflib in EL. If I'm right and you don't want to
introduce it there, you can drop everything specific for EPEL 5 packaging:

- Buildroot definition
- rm in install section
- clean section

More important things:

- Add COPYING to %doc
- Resolve the format warnings in the build
- "WARNING: unrecognized options: --enable-opt"
- Does aimage make use of ncurses and readline? It seeks for them and also
tries to link.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- 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]