Re: [ovs-dev] [PATCH v3] Don't mangle shebangs when building DKMS RPM package

2021-05-05 Thread Guzowski Adrian via dev
W dniu śro, 05.05.2021 o godzinie 13∶16 +0200, użytkownik Ilya Maximets napisał:
>
Spółka wpisana do rejestru przedsiębiorców w Sądzie Rejonowym dla m. st. 
Warszawy, XIII Wydział Gospodarczy, pod numerem KRS: 044577, kapitał 
zakładowy: 576 854 559 PLN, kapitał opłacony w całości, NIP: 527-010-45-68, 
BDO: 000250055, EXATEL SA, ul. Perkuna 47, 04-164 Warszawa.

Niniejsza wiadomość jest własnością EXATEL SA i może zawierać informacje poufne 
i/lub prawnie chronione. Jeśli nie są Państwo właściwym adresatem (lub 
otrzymali Państwo tę wiadomość na skutek pomyłki), prosimy o tym fakcie 
niezwłocznie poinformować nadawcę i usunąć otrzymaną wiadomość. Każde 
nieautoryzowane kopiowanie, ujawnianie lub rozpowszechnianie załączonej 
informacji jest zabronione.

This e-mail is a property of EXATEL SA and may contain confidential and/or 
privileged information. If you are not the intended recipient (or have received 
this e-mail by mistake) please notify the sender immediately and destroy this 
e-mail. Any unauthorized copying, disclosure or distribution of the material in 
this e-mail is strictly forbidden.
On 5/5/21 12:53 PM, Guzowski Adrian via dev wrote:
> > While building the package, some .in files are being subject to shebang
> > substitution, but the process fails, because given scripts have
> > placeholders in place of shebangs. In order to fix the issue, don't mangle
> > shebangs in this specific package.
> >
> > Signed-off-by: Guzowski Adrian 
> > ---
> >  rhel/openvswitch-dkms.spec.in | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/rhel/openvswitch-dkms.spec.in b/rhel/openvswitch-dkms.spec.in
> > index a47c038fd..5a57e4d48 100644
> > --- a/rhel/openvswitch-dkms.spec.in
> > +++ b/rhel/openvswitch-dkms.spec.in
> > @@ -27,6 +27,8 @@ BuildRoot:%(mktemp -ud 
> > %{_tmppath}/%{name}-%{version}-%{release}-
> > XX)
> >  # conflicts with the openvswitch-debuginfo package for OVS userspace).
> >  %undefine _enable_debug_packages
> >
> > +# Disable shebangs mangling
> > +%undefine __brp_mangle_shebangs
> >
> >  %description
> >  Open vSwitch Linux kernel module.
> > --
> > 2.30.2
> >
> >
> >
> > Spółka wpisana do rejestru przedsiębiorców w Sądzie Rejonowym dla m. st. 
> > Warszawy, XIII Wydział
> > Gospodarczy, pod numerem KRS: 044577, kapitał zakładowy: 576 854 559 
> > PLN, kapitał opłacony w
> > całości, NIP: 527-010-45-68, BDO: 000250055, EXATEL SA, ul. Perkuna 47, 
> > 04-164 Warszawa.
> >
> > Niniejsza wiadomość jest własnością EXATEL SA i może zawierać informacje 
> > poufne i/lub prawnie
> > chronione. Jeśli nie są Państwo właściwym adresatem (lub otrzymali Państwo 
> > tę wiadomość na
> > skutek pomyłki), prosimy o tym fakcie niezwłocznie poinformować nadawcę i 
> > usunąć otrzymaną
> > wiadomość. Każde nieautoryzowane kopiowanie, ujawnianie lub 
> > rozpowszechnianie załączonej
> > informacji jest zabronione.
> >
> > This e-mail is a property of EXATEL SA and may contain confidential and/or 
> > privileged
> > information. If you are not the intended recipient (or have received this 
> > e-mail by mistake)
> > please notify the sender immediately and destroy this e-mail. Any 
> > unauthorized copying,
> > disclosure or distribution of the material in this e-mail is strictly 
> > forbidden.
>
> Hi.  Could you, please, re-send your patches without the above
> confidentiality notice?  It makes no sense while sending emails to
> the public mail-list and it kind of conflicts with the opensource
> patch submission process.  I'm not an expert in legal aspects, but
> I think we cannot accept a patch that explicitly prohibits distribution.
>
> Best regards, Ilya Maximets.
Hello. I've requested the removal from my IT department and waiting for a 
response - this is a
preconfigured notice that every employee at our organization has.

As for the legal part, the dev mailing list is the intended recipient so there 
should not be any
issue with it, but I am too, not an expert in this matter, so if that's in fact 
a problem, I can
submit it via GitHub PR.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v3] Don't mangle shebangs when building DKMS RPM package

2021-05-05 Thread Ilya Maximets
On 5/5/21 12:53 PM, Guzowski Adrian via dev wrote:
> While building the package, some .in files are being subject to shebang
> substitution, but the process fails, because given scripts have
> placeholders in place of shebangs. In order to fix the issue, don't mangle
> shebangs in this specific package.
> 
> Signed-off-by: Guzowski Adrian 
> ---
>  rhel/openvswitch-dkms.spec.in | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/rhel/openvswitch-dkms.spec.in b/rhel/openvswitch-dkms.spec.in
> index a47c038fd..5a57e4d48 100644
> --- a/rhel/openvswitch-dkms.spec.in
> +++ b/rhel/openvswitch-dkms.spec.in
> @@ -27,6 +27,8 @@ BuildRoot:%(mktemp -ud 
> %{_tmppath}/%{name}-%{version}-%{release}-XX)
>  # conflicts with the openvswitch-debuginfo package for OVS userspace).
>  %undefine _enable_debug_packages
> 
> +# Disable shebangs mangling
> +%undefine __brp_mangle_shebangs
> 
>  %description
>  Open vSwitch Linux kernel module.
> --
> 2.30.2
> 
> 
> 
> Spółka wpisana do rejestru przedsiębiorców w Sądzie Rejonowym dla m. st. 
> Warszawy, XIII Wydział Gospodarczy, pod numerem KRS: 044577, kapitał 
> zakładowy: 576 854 559 PLN, kapitał opłacony w całości, NIP: 527-010-45-68, 
> BDO: 000250055, EXATEL SA, ul. Perkuna 47, 04-164 Warszawa.
> 
> Niniejsza wiadomość jest własnością EXATEL SA i może zawierać informacje 
> poufne i/lub prawnie chronione. Jeśli nie są Państwo właściwym adresatem (lub 
> otrzymali Państwo tę wiadomość na skutek pomyłki), prosimy o tym fakcie 
> niezwłocznie poinformować nadawcę i usunąć otrzymaną wiadomość. Każde 
> nieautoryzowane kopiowanie, ujawnianie lub rozpowszechnianie załączonej 
> informacji jest zabronione.
> 
> This e-mail is a property of EXATEL SA and may contain confidential and/or 
> privileged information. If you are not the intended recipient (or have 
> received this e-mail by mistake) please notify the sender immediately and 
> destroy this e-mail. Any unauthorized copying, disclosure or distribution of 
> the material in this e-mail is strictly forbidden.

Hi.  Could you, please, re-send your patches without the above
confidentiality notice?  It makes no sense while sending emails to
the public mail-list and it kind of conflicts with the opensource
patch submission process.  I'm not an expert in legal aspects, but
I think we cannot accept a patch that explicitly prohibits distribution.

Best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH v3] Don't mangle shebangs when building DKMS RPM package

2021-05-05 Thread Guzowski Adrian via dev
While building the package, some .in files are being subject to shebang
substitution, but the process fails, because given scripts have
placeholders in place of shebangs. In order to fix the issue, don't mangle
shebangs in this specific package.

Signed-off-by: Guzowski Adrian 
---
 rhel/openvswitch-dkms.spec.in | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/rhel/openvswitch-dkms.spec.in b/rhel/openvswitch-dkms.spec.in
index a47c038fd..5a57e4d48 100644
--- a/rhel/openvswitch-dkms.spec.in
+++ b/rhel/openvswitch-dkms.spec.in
@@ -27,6 +27,8 @@ BuildRoot:%(mktemp -ud 
%{_tmppath}/%{name}-%{version}-%{release}-XX)
 # conflicts with the openvswitch-debuginfo package for OVS userspace).
 %undefine _enable_debug_packages

+# Disable shebangs mangling
+%undefine __brp_mangle_shebangs

 %description
 Open vSwitch Linux kernel module.
--
2.30.2



Spółka wpisana do rejestru przedsiębiorców w Sądzie Rejonowym dla m. st. 
Warszawy, XIII Wydział Gospodarczy, pod numerem KRS: 044577, kapitał 
zakładowy: 576 854 559 PLN, kapitał opłacony w całości, NIP: 527-010-45-68, 
BDO: 000250055, EXATEL SA, ul. Perkuna 47, 04-164 Warszawa.

Niniejsza wiadomość jest własnością EXATEL SA i może zawierać informacje poufne 
i/lub prawnie chronione. Jeśli nie są Państwo właściwym adresatem (lub 
otrzymali Państwo tę wiadomość na skutek pomyłki), prosimy o tym fakcie 
niezwłocznie poinformować nadawcę i usunąć otrzymaną wiadomość. Każde 
nieautoryzowane kopiowanie, ujawnianie lub rozpowszechnianie załączonej 
informacji jest zabronione.

This e-mail is a property of EXATEL SA and may contain confidential and/or 
privileged information. If you are not the intended recipient (or have received 
this e-mail by mistake) please notify the sender immediately and destroy this 
e-mail. Any unauthorized copying, disclosure or distribution of the material in 
this e-mail is strictly forbidden.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev