Opened 8 years ago

Closed 5 years ago

#5680 closed defect (fixed)

./configure uses `pr`, which isn't provided by busybox

Reported by: Kylie McClain Owned by: Alexander Strasser
Priority: normal Component: build system
Version: git-master Keywords:
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

I submitted a patch to libav for this as well, but I noticed this didn't get merged into ffmpeg. Basically this patch just changes log_file() within ./configure to use printf for the formatting, rather than pr, a POSIX.1 command which is a little obscure (and therefore not provided by busybox), and easily replaceable in its usage.

Attachments (1)

0001-configure-Replace-pr-since-it-is-not-provided-by-bus.patch (1.2 KB ) - added by Kylie McClain 8 years ago.
[PATCH] configure: Replace pr since it is not provided by busybox

Download all attachments as: .zip

Change History (5)

by Kylie McClain, 8 years ago

[PATCH] configure: Replace pr since it is not provided by busybox

comment:1 by Carl Eugen Hoyos, 8 years ago

Please send your patch to the development mailing list, patches are ignored here.
Could you explain how I can reproduce on a random Linux system? (Is it possible?)

comment:2 by Carl Eugen Hoyos, 5 years ago

Version: unspecifiedgit-master

A change to logfile by Yejun Guo, based on the one attached here, was applied as ae6486c6251039d3a6bb5a90e1d818331cf55edc

comment:3 by Alexander Strasser, 5 years ago

Owner: set to Alexander Strasser
Status: newopen

comment:4 by Alexander Strasser, 5 years ago

Resolution: fixed
Status: openclosed

Should finally be fixed in commit 99147312ce6ffd3a3b70e10aacc9b64a63b6aefe .

That commit replacing pr for printing listings was not based on the patch attached to this ticket. Please refer to the commit message for details.

Note: See TracTickets for help on using tickets.