On 8/21/20 5:25 PM, Marcelo Ricardo Leitner wrote:
> On Fri, Aug 21, 2020 at 01:37:24PM -0000, GitLab Bridge on behalf of prarit 
> wrote:
>> From: Prarit Bhargava <[email protected]>
>>
>> The 'dist-configs' is not working properly as it only creates ELN
>> configs.  The 'rh-configs' and 'fedora-configs' targets are not
>> working properly and should be creating ELN and Fedora configs but they
>> only create some temporary/intermediate build files.  These targets
>> should output final .config files for each supported arch.
>>
>> There are several fixes necessary to get the *-configs targets working
>> properly:
>>
>> - Fix dist-configs to use a specified flavor instead of only ELN.  By
>> default, dist-configs will build ELN configs.
> 
> As is, dist-configs is actually redundant with rh/fedora-configs, and
> could even be removed. Well, not considering backward compatibility.
> 
> When I run it here, it:
> - generates rhel configs
> - generated fedora configs   (wasted (cpu) time)
> - process rhel configs
> 
> and final files are not tagged with 'rhel'.

That's the current way dist-configs does things.  I debated adding a rename
function to the os specific configs targets but think that should be a separate
patch.  *This* patch is to fix them so that they actually work ;)

> 
> It would be nice to have a target that generates both flavors at once.
> With that I can easily check how the config is on both and if the
> changes are getting applied correctly. But yes, I can easily script
> that around rh-configs/fedora-configs as well, 


and thus why I'm not
> seeing a reason for the dist-configs target, at least not as it is
> now. Perhaps we should hide it (from the help), as an internal target
> that should only be used to build the two other ones. Thoughts?

Not a bad idea.  dzickus?  jforbes?  I can certainly respin and remove the
'dist-configs' entry in 'make rh-help'.

> 
> rh-configs and fedora-configs are working nicely now, btw :)

P.

> 
> 
_______________________________________________
kernel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]

Reply via email to