Re: F37 proposal: RPM Macros for Build Flags (System-Wide Change proposal)

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

 



* Vít Ondruch:

> Dne 03. 06. 22 v 16:32 Tom Stellard napsal(a):
>> On 6/3/22 02:24, Vít Ondruch wrote:
>>> Hi Tom,
>>>
>>> Since you are looking into this and I like this proposal, have you
>>> considered to look alto into `%extension_*flags`:
>>>
>>> https://src.fedoraproject.org/rpms/redhat-rpm-config/blob/rawhide/f/macros#_120-123 
>>>
>>
>> I have not considered this.  Do you think there is some way this proposal
>> could be extended to help solve this problem as well?
>
>
> I think the current struggle is that we have `%extension_cflags` and
> `%build_cflags`, while we would actually need the set of
> ``%build_cflags - %extension_cflags`. The problem is that the
> `%build_cflags` order is not necessarily followed in the resulting
> code, so if we need to modify the resulting code, we have no way to
> achieve this.

I don't see how the list difference would solve your fundamental issue.
Could you elaborate?  Or maybe you already did, and you missed it, in
which case I would appreciate a pointer.

Thanks,
Florian
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux