[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 3c7a3c87d62358407d8561

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 care,

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 >expl

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 m

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 s

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 p

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 Build

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-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, >>>