Bug 225241
| Summary: | Merge Review: alsa-lib | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> |
| Component: | Package Review | Assignee: | Matthias Saou <matthias> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | stransky |
| Target Milestone: | --- | Flags: | matthias:
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: | 2007-09-20 15:18:34 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-29 20:59:56 UTC
Please find an updated spec file and a patch to the current spec file containing some suggested changes and fixes : http://thias.fedorapeople.org/merge-review/alsa-lib/ - Update License field. - Use configdir instead of sysconfdir hacks (cleaner). - Remove redundant optflags overriding. - Switch to using main "version", and merge "postver" since this is the right way of doing things (see NamingGuidelines#NonNumericRelease). - Remove static library. - Mark all of /etc/alsa as config, but not "noreplace". - Remove useless rpath on 64bit archs. Ping? It would be nice to get this package cleaned up for Fedora 8. added to rawhide. Great, thanks! I've taken the liberty to commit a last (very minor) change to remove tabs and spaces on end of lines and the dot from the summary, which obviously don't justify a rebuild. |