Opened 5 years ago
Closed 2 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)
Change History (5)
by , 5 years ago
| Attachment: | 0001-configure-Replace-pr-since-it-is-not-provided-by-bus.patch added |
|---|
comment:1 by , 5 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 , 2 years ago
| Version: | unspecified → git-master |
|---|
A change to logfile by Yejun Guo, based on the one attached here, was applied as ae6486c6251039d3a6bb5a90e1d818331cf55edc
comment:3 by , 2 years ago
| Owner: | set to |
|---|---|
| Status: | new → open |
comment:4 by , 2 years ago
| Resolution: | → fixed |
|---|---|
| Status: | open → closed |
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.
[PATCH] configure: Replace
prsince it is not provided by busybox