Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-05-03 Thread Petr Spacek
On 25.4.2016 10:12, Lukas Slebodnik wrote: > On (25/04/16 09:59), Jan Cholasta wrote: >> On 25.4.2016 09:34, Petr Spacek wrote: >>> On 25.4.2016 09:29, Lukas Slebodnik wrote: On (25/04/16 07:23), Jan Cholasta wrote: > Hi, > > On 22.4.2016 13:29, Petr Spacek wrote: >> Hello,

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-28 Thread Martin Basti
On 25.04.2016 09:39, Lukas Slebodnik wrote: On (22/04/16 13:29), Petr Spacek wrote: Hello, Makefile: add sed to BuildRequires It was requried since forever but we did not explicitly mention it. Makefile: replace perl with sed Perl was missing in BuildRequires anyway and it is used only on

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-25 Thread Lukas Slebodnik
On (25/04/16 09:59), Jan Cholasta wrote: >On 25.4.2016 09:34, Petr Spacek wrote: >> On 25.4.2016 09:29, Lukas Slebodnik wrote: >> > On (25/04/16 07:23), Jan Cholasta wrote: >> > > Hi, >> > > >> > > On 22.4.2016 13:29, Petr Spacek wrote: >> > > > Hello, >> > > > >> > > > Makefile: add sed to

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-25 Thread Jan Cholasta
On 25.4.2016 09:34, Petr Spacek wrote: On 25.4.2016 09:29, Lukas Slebodnik wrote: On (25/04/16 07:23), Jan Cholasta wrote: Hi, On 22.4.2016 13:29, Petr Spacek wrote: Hello, Makefile: add sed to BuildRequires It was requried since forever but we did not explicitly mention it. IIRC sed is

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-25 Thread Lukas Slebodnik
On (22/04/16 13:29), Petr Spacek wrote: >Hello, > >Makefile: add sed to BuildRequires > >It was requried since forever but we did not explicitly mention it. > >Makefile: replace perl with sed > >Perl was missing in BuildRequires anyway and it is used only on one place, >all other places are using

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-25 Thread Petr Spacek
On 25.4.2016 09:29, Lukas Slebodnik wrote: > On (25/04/16 07:23), Jan Cholasta wrote: >> Hi, >> >> On 22.4.2016 13:29, Petr Spacek wrote: >>> Hello, >>> >>> Makefile: add sed to BuildRequires >>> >>> It was requried since forever but we did not explicitly mention it. >> >> IIRC sed is part of the

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-25 Thread Lukas Slebodnik
On (25/04/16 07:23), Jan Cholasta wrote: >Hi, > >On 22.4.2016 13:29, Petr Spacek wrote: >> Hello, >> >> Makefile: add sed to BuildRequires >> >> It was requried since forever but we did not explicitly mention it. > >IIRC sed is part of the minimum build environemnt and as such should not be

Re: [Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-24 Thread Jan Cholasta
Hi, On 22.4.2016 13:29, Petr Spacek wrote: Hello, Makefile: add sed to BuildRequires It was requried since forever but we did not explicitly mention it. IIRC sed is part of the minimum build environemnt and as such should not be explicitly required in the spec file. I personally don't

[Freeipa-devel] [PATCH 0097-0098] Makefile: replace perl with sed

2016-04-22 Thread Petr Spacek
Hello, Makefile: add sed to BuildRequires It was requried since forever but we did not explicitly mention it. Makefile: replace perl with sed Perl was missing in BuildRequires anyway and it is used only on one place, all other places are using sed. -- Petr^2 Spacek From