Re: [OS-BUILD PATCHv2] redhat/Makefile: Add fedora-configs and rh-configs make targets

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

 



On 6/18/20 12:52 PM, Don Zickus wrote:
> On Wed, Jun 17, 2020 at 03:44:16PM -0000, GitLab Bridge on behalf of prarit wrote:
>> From: Prarit Bhargava <prarit@xxxxxxxxxx>
>>
>> There are times when I have to generate both Fedora and RHEL configs.
> 
> I believe 'make dist-configs' does this by default, builds both??
> 

That's not what I see.  From the messages:

cd /home/prarit/git-kernel/kernel-ark/redhat/configs; TARGET=fedora
./build_configs.sh kernel

where TARGET is defined by DIST_TARGET, which is

ifeq ("$(IS_FEDORA)", "1")
   DIST_TARGET=fedora
else
   DIST_TARGET=rhel
endif

IS_FEDORA, is

IS_FEDORA:=$(shell ! echo $(DIST) | grep -q fc; echo $$?)

IOW, you have to be on Fedora to get Fedora configs, and on RHEL to get RHEL
configs *or* use

make dist-configs IS_FEDORA=0 # for RHEL
make dist-configs IS_FEDORA=1 # for Fedora

which is an annoying to my doggie-sized brain :).

P.




>>
>> Add fedora-configs and rh-configs make targets.
> 
> Regardless, I like this idea and as bonus keeps muscle memory for
> rh-configs.
> 
> Acked-by: Don Zickus <dzickus@xxxxxxxxxx>
> 
>>
>> v2: Add rh-help entries.
>>
>> Signed-off-by: Prarit Bhargava <prarit@xxxxxxxxxx>
>> ---
>>  redhat/Makefile | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/redhat/Makefile b/redhat/Makefile
>> index 32542936dd1f..7cdf9e3bd0bf 100644
>> --- a/redhat/Makefile
>> +++ b/redhat/Makefile
>> @@ -141,6 +141,12 @@ dist-configs: dist-configs-prep
>>  	@cd $(REDHAT)/configs; VERSION=$(KVERSION) ./generate_all_configs.sh rhel 1; \
>>  	./process_configs.sh $(PROCESS_CONFIGS_OPTS) $(PACKAGE_NAME) $(KVERSION)
>>  
>> +fedora-configs: DIST_TARGET=fedora
>> +fedora-configs: dist-configs
>> +
>> +rh-configs: DIST_TARGET=rhel
>> +rh-configs: dist-configs
>> +
>>  dist-configs-check: dist-configs-prep
>>  	cd $(REDHAT)/configs; ./process_configs.sh $(PROCESS_CONFIGS_CHECK_OPTS) $(PACKAGE_NAME)
>>  
>> @@ -435,6 +441,8 @@ dist-full-help:
>>  	@echo  '  dist-cross-<arch>-rpms  - [x86_64 only] execute rpm builds for specified'
>>  	@echo  '                            <arch> using RHEL cross compiler.'
>>  	@echo  '                            See dist-cross-<arch>-build for the supported archs.'
>> +	@echo  '  rh-configs       - build ELN configs'
>> +	@echo  '  fedora-configs   - build Fedora configs'
>>  
>>  	@echo  ''
>>  	@echo  'kABI targets:'
>> -- 
>> GitLab
>> _______________________________________________
>> kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
>> To unsubscribe send an email to kernel-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/kernel@xxxxxxxxxxxxxxxxxxxxxxx
> 
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-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/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux