Bug 915376 - Don't need to hardcode 'Fedora'
Summary: Don't need to hardcode 'Fedora'
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: httpd
Version: 19
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Joe Orton
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-02-25 15:45 UTC by jcpunk
Modified: 2013-05-29 09:42 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-05-29 09:42:59 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
patch to make change (340 bytes, patch)
2013-02-25 15:45 UTC, jcpunk
no flags Details | Diff

Description jcpunk 2013-02-25 15:45:26 UTC
Created attachment 702399 [details]
patch to make change

Description of problem: The 'vstring' macro being set in the spec file can swapped for '%{vendor}'.  This should reduce the overhead for when the package migrates to RHEL as well as reduce the number of hard coded values within.  Since 'vstring' is the Vendor String, the macro can be used exactly as expected


Version-Release number of selected component (if applicable):httpd-2.4.3-17.fc19


How reproducible: 100%


Steps to Reproduce:
1. Open spec file
2. Notice hard coded value
  
Actual results: Vendor hard coded


Expected results: Use existing macros to avoid hard coding


Additional info:

Comment 1 Joe Orton 2013-02-26 10:37:48 UTC
Where does the "vendor" macro get defined?  On a f18 system I have:

$ rpm --eval '%{vendor}'
%{vendor}
$ rpm --eval '%{_vendor}'
redhat

Comment 2 jcpunk 2013-02-26 19:27:03 UTC
Koji is importing some macros to define it.

Grab any rpm off of http://koji.fedoraproject.org/koji/index

and run rpm -qp --qf "%{vendor}" against it.

Comment 3 Fedora End Of Life 2013-04-03 15:24:54 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.

(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora19

Comment 4 Joe Orton 2013-05-29 09:42:59 UTC
Given that:

a) vendor is undefined outside of koji we'd still need *some* default vendor string if %{vendor} is undefined, and
b) the Fedora branding is "hard-coded" into the package via index.html 

I don't see much point in making this change.


Note You need to log in before you can comment on or make changes to this bug.