Make kernel.spec friendlier to el7

[Pablo Sebastián Greco]

We rebuild the Fedora kernel for CentOS with only a few changes (mainly 
for armhfp).
Some of those changes may be part of Fedora, since they don't affect the 
build process.

disable_headers_fedora_only.patch: Fedora builds kernel-headers 
separately, do this only for fedora.
make_buildflags_optional.patch: build_cflags and build_ldflags are not 
available in el7, make them optional.

Thanks.
Pablo

[Pablo Sebastián Greco]

Didn't know about the "No attachment" rule, sorry.
disable_headers_fedora_only.patch: 
https://paste.fedoraproject.org/paste/QGmWcBvPRg3p8HnAcuNNPQ/
make_buildflags_optional.patch: 
https://paste.fedoraproject.org/paste/VWDLrz7LSRSP5k9xQZKOpg/

Pablo.

El 3/10/18 a las 15:49, Pablo Sebastián Greco escribió:
> We rebuild the Fedora kernel for CentOS with only a few changes 
> (mainly for armhfp).
> Some of those changes may be part of Fedora, since they don't affect 
> the build process.
>
> disable_headers_fedora_only.patch: Fedora builds kernel-headers 
> separately, do this only for fedora.
> make_buildflags_optional.patch: build_cflags and build_ldflags are not 
> available in el7, make them optional.
>
> Thanks.
> Pablo
>
> _______________________________________________
> kernel mailing list -- kernel(a)lists.fedoraproject.org
> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: 
> https://lists.fedoraproject.org/archives/list/[e-mail]

[Peter Robinson]

On Wed, Oct 3, 2018 at 8:12 PM Pablo Sebastián Greco
<pablo(a)fliagreco.com.ar&gt; wrote:
>  Didn't know about the "No attachment" rule, sorry. 
The general rule is use "git send-email" and have them inline.

> disable_headers_fedora_only.patch:
> https://paste.fedoraproject.org/paste/QGmWcBvPRg3p8HnAcuNNPQ/
> make_buildflags_optional.patch:
> https://paste.fedoraproject.org/paste/VWDLrz7LSRSP5k9xQZKOpg/
>
> Pablo.
>
> El 3/10/18 a las 15:49, Pablo Sebastián Greco escribió:
> > We rebuild the Fedora kernel for CentOS with only a few changes
> > (mainly for armhfp).
> > Some of those changes may be part of Fedora, since they don't affect
> > the build process.
> >
> > disable_headers_fedora_only.patch: Fedora builds kernel-headers
> > separately, do this only for fedora.
> > make_buildflags_optional.patch: build_cflags and build_ldflags are not
> > available in el7, make them optional.
> >
> > Thanks.
> > Pablo
> >
> > _______________________________________________
> > kernel mailing list -- kernel(a)lists.fedoraproject.org
> > To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
> > Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
> > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> > List Archives:
> > https://lists.fedoraproject.org/archives/list/[e-mail]
> _______________________________________________
> kernel mailing list -- kernel(a)lists.fedoraproject.org
> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
https://lists.fedoraproject.org/archives/list/[e-mail]

[Josh Boyer]

On Wed, Oct 3, 2018 at 4:13 PM Peter Robinson <pbrobinson(a)gmail.com&gt; wrote:
>  On Wed, Oct 3, 2018 at 8:12 PM Pablo Sebastián Greco
>  <pablo(a)fliagreco.com.ar&gt; wrote:
>  >
>  > Didn't know about the "No attachment" rule, sorry.
>
>  The general rule is use "git send-email" and have them inline. 
You could submit a pull request with pagure too.

josh

[Pablo Sebastián Greco]

El 3/10/18 a las 17:12, Peter Robinson escribió:
>  On Wed, Oct 3, 2018 at 8:12 PM Pablo Sebastián Greco
>  <pablo(a)fliagreco.com.ar&gt; wrote:
> > Didn't know about the "No attachment" rule, sorry.
>  The general rule is use "git send-email" and have them inline. 
Ack,
I'm learning things as I go ;)
>
>> disable_headers_fedora_only.patch:
>> https://paste.fedoraproject.org/paste/QGmWcBvPRg3p8HnAcuNNPQ/
>> make_buildflags_optional.patch:
>> https://paste.fedoraproject.org/paste/VWDLrz7LSRSP5k9xQZKOpg/
>>
>> Pablo.
>>
>> El 3/10/18 a las 15:49, Pablo Sebastián Greco escribió:
>>> We rebuild the Fedora kernel for CentOS with only a few changes
>>> (mainly for armhfp).
>>> Some of those changes may be part of Fedora, since they don't affect
>>> the build process.
>>>
>>> disable_headers_fedora_only.patch: Fedora builds kernel-headers
>>> separately, do this only for fedora.
>>> make_buildflags_optional.patch: build_cflags and build_ldflags are not
>>> available in el7, make them optional.
>>>
>>> Thanks.
>>> Pablo
>>>
>>> _______________________________________________
>>> kernel mailing list -- kernel(a)lists.fedoraproject.org
>>> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
>>> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
>>> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>>> List Archives:
>>> https://lists.fedoraproject.org/archives/list/[e-mail]
>> _______________________________________________
>> kernel mailing list -- kernel(a)lists.fedoraproject.org
>> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
>> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
>> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>> List Archives:
https://lists.fedoraproject.org/archives/list/[e-mail]
> _______________________________________________
> kernel mailing list -- kernel(a)lists.fedoraproject.org
> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
https://lists.fedoraproject.org/archives/list/[e-mail]

[Pablo Sebastián Greco]

El 3/10/18 a las 17:16, Josh Boyer escribió:
>  On Wed, Oct 3, 2018 at 4:13 PM Peter Robinson
> <pbrobinson(a)gmail.com&gt; wrote:
> > On Wed, Oct 3, 2018 at 8:12 PM Pablo Sebastián Greco
> > <pablo(a)fliagreco.com.ar&gt; wrote:
> >> Didn't know about the "No attachment" rule, sorry.
> > The general rule is use "git send-email" and have them inline.
>  You could submit a pull request with pagure too. 
(12:28:18) pgreco: labbott, paste
here or do you prefer a PR?
(12:32:21) labbott: I'd prefer patches sent to the 
kernel(a)lists.fedoraproject.org <mailto:[e-mail]> 
mailing list but a PR is okay
>  josh
>

[PATCH 1/2] Disable kernel-headers only in Fedora

From: Pablo Greco <pablo(a)fliagreco.com.ar&gt;

---
 kernel.spec | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/kernel.spec b/kernel.spec
index 311ef4c..e90a714 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -125,8 +125,10 @@ Summary: The Linux kernel
 %define debugbuildsenabled 0
 
 # Kernel headers are being split out into a separate package
+%if 0%{?fedora}
 %define with_headers 0
 %define with_cross_headers 0
+%endif
 
 %if %{with_verbose}
 %define make_opts V=1
-- 
1.8.3.1

[PATCH 2/2] Make buildflags optional

From: Pablo Greco <pablo(a)fliagreco.com.ar&gt;

---
 kernel.spec | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel.spec b/kernel.spec
index e90a714..aadae04 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -1204,8 +1204,8 @@ cp_vmlinux()
 # from redhat-rpm-config assume that host == target so target arch
 # flags cause issues with the host compiler.
 %if !%{with_cross}
-%define build_hostcflags  %{build_cflags}
-%define build_hostldflags %{build_ldflags} -Wl,--build-id=uuid
+%define build_hostcflags  %{?build_cflags}
+%define build_hostldflags %{?build_ldflags} -Wl,--build-id=uuid
 %endif
 
 BuildKernel() {
-- 
1.8.3.1

[PATCH 1/2] Disable kernel-headers only in Fedora

[Laura Abbott]

On 10/11/2018 05:58 AM, pablo(a)fliagreco.com.ar wrote:
>  From: Pablo Greco <pablo(a)fliagreco.com.ar&gt;
>  
>  ---
>    kernel.spec | 2 ++
>    1 file changed, 2 insertions(+)
>  
>  diff --git a/kernel.spec b/kernel.spec
>  index 311ef4c..e90a714 100644
>  --- a/kernel.spec
>  +++ b/kernel.spec
>  @@ -125,8 +125,10 @@ Summary: The Linux kernel
>    %define debugbuildsenabled 0
>    
>    # Kernel headers are being split out into a separate package
>  +%if 0%{?fedora}
>    %define with_headers 0
>    %define with_cross_headers 0
>  +%endif
>    
>    %if %{with_verbose}
>    %define make_opts V=1
>   
Thanks, I applied both of the patches to rawhide.