Bug 3957 – Busybox does not support "ls -Q" which some scripts need

Bug 3957 - (int108362/int-113308) Busybox does not support "ls -Q" which some scripts need

(int108362/int-113308)
Summary:	Busybox does not support "ls -Q" which some scripts need

Status:	RESOLVED FIXED

Product:	Core
Component:	Busybox
Version:	5.0-beta
Platform:	All Maemo

Importance:	Low normal with 1 vote (vote)
Target Milestone:	Harmattan
Assigned To:	Turo Janka
QA Contact:	busybox-bugs

URL:	https://bugs.busybox.net/show_bug.cgi...
Keywords:	community-diablo, patch, upstream

Depends on:
Blocks:	3938 3948
	Show dependency tree

Reported:	2008-12-23 23:02 UTC by tz
Modified:	2010-03-21 16:33 UTC (History)
CC List:	3 users (show)

See Also:

Attachments
Upstream patch (just the -Q part) backported to 1.10.2 (2.56 KB, patch) 2009-06-27 11:11 UTC, Lucas Maneos	Details
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description tz (reporter) 2008-12-23 23:02:57 UTC

SOFTWARE VERSION:
(Control Panel > General > About product)
5.2008.43-7

STEPS TO REPRODUCE THE PROBLEM:
enter "ls -Q" on the tablet (v.s. under scratchbox)

EXPECTED OUTCOME:
quoted list of files.

ACTUAL OUTCOME:
usage message with a list of valid options

REPRODUCIBILITY:
(always/sometimes/once)
always

EXTRA SOFTWARE INSTALLED:

OTHER COMMENTS:
This is not a "posix standard" option, but does occur in /usr/bin/invoke-rc.d
used by debian pre/post inst/rm scripts for things in /etc/init.d

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.5)
Gecko/2008120122 Firefox/3.0.5

Comment 1 Andre Klapper maemo.org

2009-01-15 13:20:41 UTC

I've filed this upstream: https://bugs.busybox.net/show_bug.cgi?id=49

Comment 2 Andre Klapper maemo.org

2009-03-03 17:33:52 UTC

Hi Turo, 
an upstream patch is available at https://bugs.busybox.net/attachment.cgi?id=87
and has been committed as
http://sources.busybox.net/index.py?view=rev&revision=25505 .
Can you integrate this into the 1.10.x version shipped for Fremantle?

Comment 3 Eero Tamminen nokia

2009-04-21 11:57:46 UTC

"ls" in newly released BusyBox 1.14.0 supports "-Q" option:
  http://www.busybox.net/

Comment 4 Andre Klapper maemo.org

2009-04-27 16:09:49 UTC

Forwarding "WONTFIX for Fremantle" comment from internal ticket:
"While "ls" in BusyBox 1.14.0 supports "-Q" option, backporting it to 1.10
would mean implementing it from scratch which will involve reshuffling of other
ls options. Which is not a worthy target in my opinion. Do we have a rule in
the policy or something for fremantle which warns against using -Q? Would be
nice to have."

Keeping this ticket opened for Harmattan.

Comment 5 Lucas Maneos 2009-06-27 11:11:18 UTC

Created an attachment (id=1250) [details]
Upstream patch (just the -Q part) backported to 1.10.2

(In reply to comment #4)
> Forwarding "WONTFIX for Fremantle" comment from internal ticket:
> "While "ls" in BusyBox 1.14.0 supports "-Q" option, backporting it to 1.10
> would mean implementing it from scratch which will involve reshuffling of
> other ls options.

It's not quite that bad - at first sight it looks like there's no space for
more options in 1.10.2's flags bitmap, but we can squeeze one more in by
changing the opt_flags terminating value to something that can't possibly be an
option.

Comment 6 Lucas Maneos 2009-10-22 07:57:39 UTC

Marking patches of interest to Diablo (Maemo4) community updates, please excuse
the noise.

Comment 7 Andre Klapper maemo.org

2009-12-10 16:39:11 UTC

This has been fixed for Harmattan/Maemo6.
For Maemo5, this is a WONTFIX.

Comment 8 Lucas Maneos 2010-03-21 16:33:50 UTC

FYI, there's an issue remaining where the column width on multi-column output
isn't adjusted properly to account for the quotes. eg:

$ touch a b
$ ls -Q 
"a""b"

Sent one-liner fix to upstream bug against 1.16.0, shouldn't be hard to apply
to whatever version will go into Harmattan.