Bug 295941
| Summary: | Add all_alpha macro to macros | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Oliver Falk <oliver> |
| Component: | rpm | Assignee: | Panu Matilainen <pmatilai> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | low | Docs Contact: | |
| Priority: | low | ||
| Version: | rawhide | CC: | pnasrat |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-10-03 11:17:29 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
Oliver Falk
2007-09-19 08:02:48 UTC
Adding something like that is not a problem as such, what bugs me is that the
"all" macros are such a hodgepodge naming-wise: %{ix86} for all x86, %{arm} for
all arm processors and now %{all_alpha}...
Oh well, the old names can't be changed without breaking half the worlds specs
relying on them, new ones can of course be added. I'd actually prefer %alpha_all
to keep it in alpha* "namespace" if that's ok?
We cannot use simple %alpha, can we? I'm not sure... ifarch alpha and ifarch %alpha would be different then!? But someone could over read that... Maybe stick with %alpha_all :-) That's fine for me! %alpha is possible, sure, just potentially somewhat confusing as like you say, "%ifarch alpha" vs "%ifarch %alpha" are then entirely different, but then so is "patch" and "%patch"... Then we use %alpha. The problem persists with other macro/non-macro definitions as well. Using %alpha only we have the same naming as x86, arm, ... OK Panu? Done deal :) Upstream now, will be in 4.4.2.2 when released -> in Fedora after test3-freeze. In rawhide now (but only available for builders atm due to freeze) That's fine. Thx. |