Hi,
attached patch replaces two perl calls with sed, and adds an alternative for another two (using GNU date, and with limited impact if both alternatives fail)
Those were the remaining perl calls in our build tree, so remove perl from our dependency list in README.
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
On 2/10/10 1:24 PM, Patrick Georgi wrote:
Hi,
attached patch replaces two perl calls with sed, and adds an alternative for another two (using GNU date, and with limited impact if both alternatives fail)
Those were the remaining perl calls in our build tree, so remove perl from our dependency list in README.
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Acked-by: Stefan Reinauer stepan@coresystems.de
On Wed, Feb 10, 2010 at 01:24:03PM +0100, Patrick Georgi wrote:
Hi,
attached patch replaces two perl calls with sed, and adds an alternative for another two (using GNU date, and with limited impact if both alternatives fail)
Looks great, but why not remove the perl line completely and use date only? I think there's no benefit of keeping the perl alternative, correct?
Uwe.
Am 10.02.2010 19:06, schrieb Uwe Hermann:
Looks great, but why not remove the perl line completely and use date only? I think there's no benefit of keeping the perl alternative, correct?
This way, it still works with non-GNU date, if perl is installed. My main point of removing perl is to support windows better (smaller toolchain required), and there, date is GNU. On most unix-alikes, either GNU date or perl are around.
Patrick