Bug 7701 - Separator of item in display_finfo() should be Tab
Separator of item in display_finfo() should be Tab
Status: NEW
Product: Samba 3.3
Classification: Unclassified
Component: Client tools
3.3.1
Other Linux
: P3 normal
: ---
Assigned To: Volker Lendecke
Samba QA Contact
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-09-28 01:21 UTC by H Hasegawa
Modified: 2010-11-05 04:11 UTC (History)
1 user (show)

See Also:


Attachments
Patch for 3.x (include bug 7700 and 7701) (3.37 KB, patch)
2010-10-28 02:45 UTC, Brandon
no flags Details
Patch for v3-6-test (1.17 KB, patch)
2010-11-05 04:11 UTC, Brandon
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description H Hasegawa 2010-09-28 01:21:11 UTC
Hi, 

Thank you for your continued business.
I line 508 of display_finfo() of client.c is not suitable for recognizing output of file list.
The separator of d_printf() is space but filename often includes space. 
If the last character of filename was space, then we couldn't recognize it as a part of filename or separator.
It might be good if the separator is \t, not space.
Thank you for your attention.
Best regards
Comment 1 Volker Lendecke 2010-09-28 01:24:15 UTC
Your description of the problems (also bug 7700) is very concrete, almost a plaintext description of the patch. It would probably make it quicker to get the changes in if you uploaded them as a patch. See

http://wiki.samba.org/index.php/Using_Git_for_Samba_Development

for a quick description to create such a patch.

Thanks,

Volker
Comment 2 H Hasegawa 2010-09-28 01:35:37 UTC
Hi,

Thank you for your advice.
All right. I'll study it and create patch.
Thanks a lot.
Best regards
Comment 3 Brandon 2010-10-28 02:45:02 UTC
Created attachment 6038 [details]
Patch for 3.x (include bug 7700 and 7701)

A patch for solving the problems 7700 and 7701.
Comment 4 Volker Lendecke 2010-11-02 16:21:28 UTC
Against which tree did you create this patch? It does not apply to any of the v3-x-test branches we right now have open.

Thanks,

Volker
Comment 5 Brandon 2010-11-04 02:29:33 UTC
I am so sorry for late reply. Please forgive 
me that I have made a incorrect descriptions. 
The against branch is "master", but not any 
v3-x-test. 
 
I have a question, does it need to make a 
new one for v3-5-test? How about the version 
v3-6-test?

Thanks,

Best,
Brandon
Comment 6 Volker Lendecke 2010-11-04 04:08:07 UTC
Oh, that is interesting. Your patch contains the two lines

-void do_list(const char *mask,
+int do_list(const char *mask,

The - line usually means that this line is about to be deleted. When I look at current master, it contains a line

NTSTATUS do_list(const char *mask,

My implementation of the "patch" program is not able to delete the line "void do_list...", because it finds a line "NTSTATUS do_list...".

What version of "patch" are you using to cleanly apply this patch?

Thanks,

Volker
Comment 7 Brandon 2010-11-04 19:45:07 UTC
Thank you for reply,

That means I have an old version of the source of samba? I have 
got them from last month. Until now, I have not updated the latest 
version. Has someone else improved the problem already during 
these days? All right, I will update the latest version for checking. 
Add after that, I will offer a new patch.

We hope that the following versions will be applied for the patch.
  origin/v3-3-*  (presently used)
  origin/v3-5-*  (be used soon)
Would you like to tell us if we want to use the upgraded versions 
such as "origin/v3-6-*" and "origin/v4-0-*", we must offer the new 
patchs for them respectively?

Besides, we are familiar with "SVN", but "git" is new for us. I'd 
like to study it. Could you tell me the following steps are correct? 
    $ git clone git://git.samba.org/samba.git samba
    $ git checkout -b my_v3-3-test origin/v3-3-test
Then, we do our changes, 
after that,
    $ git add my_v3-3-test
    $ git commit -m "7701-xxx-yyy-zzz"
Finally, we make the patch by, 
    $ git format-patch -1

Thanks,

Best 
Brandon.
Comment 8 Brandon 2010-11-05 04:11:02 UTC
Created attachment 6047 [details]
Patch for v3-6-test

The local sources have been updated today.
Found that the bug 7700 has been fixed partly 
except for "cmd_mkdir()"" and "cmd_rmdir()". 

Therefore, the new patch has been made for the 
two remained functions and bug7701. The agaist 
branch is "origin/v3-6-test".

Thanks,

Best,
Brandon