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

[Guzowski Adrian]

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: 0000044577, 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 <adrian.guzowski at exatel.pl>
> > ---
> >  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}-
> > XXXXXX)
> >  # 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: 0000044577, 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.