[PATCH] attrib: Move implementation from cmd.exe to the standalone command (try 4) (resend)

Frédéric Delanoy frederic.delanoy at gmail.com
Tue Sep 6 08:33:27 CDT 2011


On Tue, Sep 6, 2011 at 09:35, Christian Costa <titan.costa at wanadoo.fr> wrote:
> --
>
> It's just a resend. The patch is unmodified, just rebased on latest git.
>
> try4:
> - add missing NULL ending character to help_option string
>
> try3:
> - fix /? option. This fixes cmd tests regression
>
> try2:
> - keep attrib infos for help command
> - update po files
>
> I added a stub last year to make an installer work but didn't know the command existed as a builtin in cmd.exe until Frédéric Delanoy told me. So here is a patch that takes what's in cmd.exe and move it to the standalone command. Some few code was taken from xcopy.

I've got test failures with your patch ("make test" in programs/cmd/tests)
First error:
batch.c:301: Test failed: unexpected char 0x6e position 7 in line 591
(got 'r.test not found before delete, bad', wanted 'r.test found
before delete, good')

Also, beware that testbot won't test your patch on windowses since it
doesn't change any file in tests/ directory
(you might want to submit it manually with, say, a small change in
test_builtins.cmd and/or test_builtins.exp)

See http://wiki.winehq.org/CmdConformanceTests (recently written) if
you need help with the keywords, etc.

Frédéric



More information about the wine-devel mailing list