darcs

Issue 456 'darcs query manifest' doesn't show managed empty folders

Title 'darcs query manifest' doesn't show managed empty folders
Priority bug Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, giksos, kowey, thorkilnaur, tommy
Assigned To kowey
Topics FauxBug

Created on 2007-05-10.03:01:32 by giksos, last changed 2009-08-27.14:11:46 by admin.

Messages
msg1682 (view) Author: giksos Date: 2007-05-10.03:01:31
Here is the problem:

=========================
:~/tmp/darcs> darcs --version
1.0.8 (release)
:~/tmp/darcs> ls
:~/tmp/darcs> darcs init
:~/tmp/darcs> mkdir dir
:~/tmp/darcs> darcs add dir
:~/tmp/darcs> darcs query manifest
:~/tmp/darcs>
=========================

As you see, the folder "dir" is actually managed by darcs, but is not shown in
the manifest.
msg1684 (view) Author: tommy Date: 2007-05-10.07:01:43
Check out 'darcs help query manifest' and notice that
--no-directories is the default.
msg1832 (view) Author: kowey Date: 2007-07-16.00:53:38
Works with darcs query manifest --directories (as Tommy pointed out).  On the
other hand, it might be fair to say that this should the default, not the other
way around.
msg1835 (view) Author: tommy Date: 2007-07-16.09:18:41
> other hand, it might be fair to say that this should the default, not the other

This was suggested when the manifest command was implemented.

Ian Lynagh:
| Default to listing everything is probably simplest.

Florian Weimer (implementer of the manifest command):
| Usually, a manifest file does not list directories, doesn't it?
| Therefore, I've made --no-directories the default.  But I can change
| this easily.

Simplicity and least-surprise are of course good reasons for
changing it, but there have not been a great deal of reported
surprise (compared to other things with darcs that we have
changed because of that reason), and there might be users who
think that manifest generating a "normal" manifest is simpler. I
don't use the command myself so I don't have a strong opinion,
but maybe we should propose such a change on the user mailing
list first.
msg1866 (view) Author: kowey Date: 2007-07-18.20:52:39
Just to follow up, the advice on darcs-users has been to rename 'query manifest'
to 'list files' (query is retained as alias for list; and manifest as an alias
for files).  List files will show directories by defaults; manifest will retain
the old behaviour.

Will submit a patch for comments.
msg1903 (view) Author: kowey Date: 2007-07-23.07:41:25
Resolved in unstable with the following patches.  Thanks!

Tue Jul 17 00:03:08 CEST 2007  Eric Kow <eric.kow@loria.fr>
  * Make --directories the default in query manifest (issue456).

Wed Jul 18 22:47:03 CEST 2007  Eric Kow <eric.kow@loria.fr>
  * Rename query to list; manifest to files.
  
    query          => list (with query as alias)
    query manifest => list files (with manifest as alias)
    query tags     => list tags
  
  Note that the list manifest and list files commands differ in that the
  former displays directories by default and the latter does not.

The naming of the commands is still not settled, but we're working on it
(darcs-users).
History
Date User Action Args
2007-05-10 03:01:32giksoscreate
2007-05-10 07:01:56tommysetstatus: unread -> unknown
nosy: droundy, tommy, beschmi, kowey, giksos
messages: + msg1684
2007-07-16 00:53:43koweysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, giksos
topic: + FauxBug
messages: + msg1832
2007-07-16 09:18:42tommysetstatus: resolved -> unknown
messages: + msg1835
2007-07-18 20:52:41koweysetmessages: + msg1866
2007-07-23 07:41:27koweysetstatus: unknown -> resolved-in-unstable
messages: + msg1903
2007-07-31 17:58:08koweysetstatus: resolved-in-unstable -> resolved-in-stable
2008-09-16 21:31:03adminsetstatus: resolved-in-stable -> resolved
nosy: + dagit
2009-08-06 17:42:47adminsetnosy: + markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy, giksos
2009-08-06 20:39:44adminsetnosy: - beschmi
2009-08-10 22:04:08adminsetnosy: + giksos, - markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-11 00:00:29adminsetnosy: - dagit
2009-08-25 17:56:07adminsetnosy: + darcs-devel, - simon
2009-08-27 14:11:46adminsetnosy: tommy, kowey, darcs-devel, giksos, thorkilnaur, dmitry.kurochkin