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 one place,
all other places are using sed.

--
Petr^2 Spacek
>From 3c7a3c87d62358407d856119e384c70040acec1e Mon Sep 17 00:00:00 2001
From: Petr Spacek <pspa...@redhat.com>
Date: Fri, 22 Apr 2016 10:40:11 +0200
Subject: [PATCH] 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.
---
Makefile | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

ACK

>From 2deaef91ab71c0e78b2142c2102cfe65f0e4ed96 Mon Sep 17 00:00:00 2001
From: Petr Spacek <pspa...@redhat.com>
Date: Fri, 22 Apr 2016 10:40:37 +0200
Subject: [PATCH] Makefile: add sed to BuildRequires

It was requried since forever but we did not explicitly mention it.
---
freeipa.spec.in | 1 +
1 file changed, 1 insertion(+)

NACK

As it was mentioned elsewhere in thread this patch is not needed.

LS

pushed to master:
* 8689e6be515aaf5b3d6c891e9e450b99cd8af8d4 Makefile: replace perl with sed

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to