Re: new ls option: --group-directories-first

Jim Meyering ha scritto: Hi Francesco, I've checked in your changes, with minor syntactic changes, and will add test cases next week. Thank you for contributing that. Thanks indeed! And thanks also to Eric Blake who guided me with precious advices to make this patch ! Just one little

Re: new ls option: --group-directories-first

Jim Meyering ha scritto: Just one little question about this patch: are you sure not to add a short option for --group-directories-first ? For now, yes. It would take a strong argument to go against the `no new short option names' policy for ls, especially considering the alternative

Re: ls --group-directories-first

Francesco Montorsi ha scritto: Jim Meyering ha scritto: Francesco Montorsi [EMAIL PROTECTED] wrote: ... Finally copyright stuff should be okay. So, I repost the patch ;) Thanks for the ping. There is at least one patch in the queue ahead of this. One thing you can do to accelerate the process

Re: ls --group-directories-first

Francesco Montorsi ha scritto: Jim Meyering ha scritto: Francesco Montorsi [EMAIL PROTECTED] wrote: anyone on this ? Francesco Montorsi ha scritto: Hi, I've got the reply from FSF copyright ! Finally copyright stuff should be okay. So, I repost the patch ;) Thanks for the ping

Re: ls --group-directories-first

Jim Meyering ha scritto: Francesco Montorsi [EMAIL PROTECTED] wrote: ... Finally copyright stuff should be okay. So, I repost the patch ;) Thanks for the ping. There is at least one patch in the queue ahead of this. One thing you can do to accelerate the process is to add a few tests. ok

Re: ls --group-directories-first

Jim Meyering ha scritto: Francesco Montorsi [EMAIL PROTECTED] wrote: anyone on this ? Francesco Montorsi ha scritto: Hi, I've got the reply from FSF copyright ! Finally copyright stuff should be okay. So, I repost the patch ;) Thanks for the ping. There is at least one patch in the queue

Re: Final patch version (was Re: [patch #3596] Sort directories before files in ls)

anyone on this ? Francesco Montorsi ha scritto: Hi, I've got the reply from FSF copyright ! Finally copyright stuff should be okay. So, I repost the patch ;) Francesco ChangeLog changes: 2005-12-27 Francesco Montorsi [EMAIL PROTECTED] Add a --group-directories-first option

Re: Final patch version (was Re: [patch #3596] Sort directories before files in ls)

Hi, I've got the reply from FSF copyright ! Finally copyright stuff should be okay. So, I repost the patch ;) Francesco ChangeLog changes: 2005-12-27 Francesco Montorsi [EMAIL PROTECTED] Add a --group-directories-first option to ls to group directories before files

Re: Final patch version (was Re: [patch #3596] Sort directories before files in ls)

Hi, do we really have to wait for FSF copyright paperwork before applying this patch ? I don't have any info from [EMAIL PROTECTED] since some time. I fear that this patch could get lost... Thanks, Francesco ___ Bug-coreutils mailing list

Final patch version (was Re: [patch #3596] Sort directories before files in ls)

be used for such kind of check. The patch should now be good enough, in any case I'm ready to fix other small issues. I'm still waiting for FSF copyright paperwork... I'll let you know when they send it to me. Francesco ChangeLog changes: 2005-12-27 Francesco Montorsi [EMAIL PROTECTED

Re: [patch #3596] Sort directories before files in ls

have defined num_sorts but then left out the ARGMATCH_VERIFY check as you can see below... Can you take a look at this ? Thanks, Francesco ChangeLog changes: 2005-12-27 Francesco Montorsi [EMAIL PROTECTED] Add a --group-directories-first option to ls to group directories

[patch #3596] Sort directories before files in ls

Additional Item Attachment, patch #3596 (project coreutils): File name: changelog.patchSize:0 KB Changelog patch http://savannah.gnu.org/patch/download.php?item_id=3596item_file_id=5665 ___ Reply to this item at:

Re: [patch #3596] Sort directories before files in ls

Hi, Eric Blake wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Francesco Montorsi on 12/22/2005 1:41 AM: EVIL. You are calling qsort twice for every element (once to put directories first, then twice again on the subsets). This is twice as slow as just writing a proper

Re: [patch #3596] Sort directories before files in ls

Alfred M. Szmidt wrote: sure; the patch (int its old version - see Eric blake comments on it) is attached to this mail. I wanted to look at the updated one, sorry for not being clear. sorry but before updating the patch I'd like to have a reply to my following problem: Eric Blake

Re: [patch #3596] Sort directories before files in ls

. + * src/ls.c: Added the -e,--group-directories option to group + directories before files (Francesco Montorsi). * src/rm.c (long_opts): Change the name of each undocumented, for- testing-only option to start with `-', so that it cannot render Index: NEWS

Re: [patch #3596] Sort directories before files in ls

Any comments on my last comments ;) ? Francesco ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils

Re: [patch #3596] Sort directories before files in ls

Hi, I've compiled and sent it... Thanks, Francesco Jim Meyering wrote: Francesco Montorsi [EMAIL PROTECTED] wrote: ... PS: as Eric wrote in another mail, could you please send me the paperwork I will need to mail into the FSF ? Sure. Here's the form: Please email

[patch #3596] Sort directories before files in ls

Follow-up Comment #4, patch #3596 (project coreutils): PS: a couple of things I forgot: 1) I added another small patch which modifies also the coreutils.texi file 2) I assign the copyright to the FSF for all my patches submitted in this tracker

[patch #3596] Sort directories before files in ls

Follow-up Comment #2, patch #3596 (project coreutils): If the original patch author doesn't mind, I'll try to make the changes suggested by Eric Blake and then make a new patch. I'll let you know. ___ Reply to this item at:

[patch #3596] Sort directories before files in ls

Follow-up Comment #3, patch #3596 (project coreutils): I've added to this tracker a new patch which should address all issues raised by Eric Blake. The only thing left out are systems without dirent.d_type: a stat for each file isn't going to be too slow ? Maybe that for those systems without

Re: Feature request: add option to show directories first

Arg, I see that a patch for this option was already done ! http://savannah.gnu.org/patch/?func=detailitemitem_id=3596 it was submitted on 26/12/2004; did anyone ever looked at this ? Francesco ___ Bug-coreutils mailing list Bug-coreutils@gnu.org

Feature request: add option to show directories first

: C A B Do you agree ? Francesco Montorsi ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils

Re: Feature request: add option to show directories first

Hi, Eric Blake wrote: [The fact that you sent this request to the old bug-fileutils list instead of bug-coreutils makes me wonder if you have the latest version of coreutils. However, your request is still not present in CVS coreutils.] sorry; I did not know that bug-fileutils was