Bug 480886
Summary: | Review Request: tse3 - MIDI Sequencer Engine | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Orcan Ogetbil <oget.fedora> | ||||
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | bugs.michael, fedora-package-review, mtasaka, notting | ||||
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
kevin: fedora-cvs+ |
||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2009-01-27 17:07:05 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: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 480887 | ||||||
Attachments: |
|
Description
Orcan Ogetbil
2009-01-21 04:41:36 UTC
Created attachment 329952 [details]
some trial patch
For 0.3.1-1:
* About Patch2:
- Well, google shows that awe_voice.h was removed from
kernel headers at 2.6.21 or so (i.e. around Fedora 7)
I don't think it is preferrable to re-introduce macros
which was defined in the header file which is no longer
supported. I guess kernel supports AWE32 in other way
or kernel does not support AWE32 completely any more.
Are macros related to awe_voice.h really needed even
now?
(removing all awe32 related parts like attached does not help ?
I just tried to remove all awe_voice.h related parts
and did not any functionality check as I don't know
how to use this software...)
* Timestamps
- Please consider to use
-------------------------------------------------------
make install DESTDIR=%{buildroot} INSTALL="install -p"
-------------------------------------------------------
to keep timestamps on installed files.
* Directory structure about document files
- For HTML files in devel package, I suggest that a subdirectory
should be created under %_defaultdocdir/%name-devel-%version
(named as "HTML", for example) and all HTML files should be
moved into the directory.
Thank you. (In reply to comment #1) > > * About Patch2: > - Well, google shows that awe_voice.h was removed from > kernel headers at 2.6.21 or so (i.e. around Fedora 7) > I don't think it is preferrable to re-introduce macros > which was defined in the header file which is no longer > supported. I guess kernel supports AWE32 in other way > or kernel does not support AWE32 completely any more. > > Are macros related to awe_voice.h really needed even > now? > (removing all awe32 related parts like attached does not help ? > I just tried to remove all awe_voice.h related parts > and did not any functionality check as I don't know > how to use this software...) > Thanks for the patch. It seems like you are right. But now I am thinking of dropping the OSS support from tse3 altogether. I see other applications/libraries in Fedora built with --without-oss flag. - It will save us some dependencies, - I am not sure how functional the library will be after this patch, and - OSS is deprecated, so we won't loose much. Does that sound OK? I think removing OSS support should be okay on Fedora. Update: Spec URL: http://oget.fedorapeople.org/review/tse3.spec SRPM URL: http://oget.fedorapeople.org/review/tse3-0.3.1-2.fc10.src.rpm Changelog: - Move the HTML documentation into an HTML subdirectory - Use INSTALL="install -p" - Drop the oss support, and the related patch Please provide srpm, not binary rpm :) Uh oh. Sorry, my bad! I uploaded the appropriate SRPM file to the same place. Okay, good. ----------------------------------------------------- This package (tse3) is APPROVED by mtasaka ----------------------------------------------------- Great. Thanks! New Package CVS Request ======================= Package Name: tse3 Short Description: MIDI Sequencer Engine Owners: oget Branches: F-9 F-10 InitialCC: cvs done. > Requires: %{name} == %{version}-%{release} Does that work? > %install > make install DESTDIR=%{buildroot} INSTALL="install -p" \ "rm -rf %{buildroot}" is missing as the first cmd in %install section. (In reply to comment #10) > > Requires: %{name} == %{version}-%{release} > > Does that work? Yes (at least on F-9/10/11) > > %install > > make install DESTDIR=%{buildroot} INSTALL="install -p" \ > > "rm -rf %{buildroot}" is missing as the first cmd in %install section. Sorry for not pointing this out... Closing. (In reply to comment #11) > (In reply to comment #10) > > > %install > > > make install DESTDIR=%{buildroot} INSTALL="install -p" \ > > > > "rm -rf %{buildroot}" is missing as the first cmd in %install section. > > Sorry for not pointing this out... No problem. I missed this as much as you did. But this is weird because rpmlint normally warns us if one of the "rm -rf %{buildroot}" is missing (one in %install, one in %clean). Maybe we hit an rpmlint bug. I'm revoking my requests for pushing to F-9 and F-10 update-testing and rebuilding. |