Incorrect path separator used in testdir/Make_ming.mak

9 views
Skip to first unread message

Michael Soyka

unread,
Sep 13, 2020, 12:24:16 PM9/13/20
to vim_dev
Vim developers,

The following statement

    if not exist $@ $(DEL) $@

on line 98 in testdir/Make_ming.mak has two errors:

1.  it attempts to delete a non-existing file and
2.  the path that is substituted for $@ includes unix/linux path separators.

The attached patch fixes both. 

Regards,

-mike
0001-Fix-path-separator-in-Make_ming.mak.patch

Bram Moolenaar

unread,
Sep 13, 2020, 3:04:27 PM9/13/20
to vim...@googlegroups.com, Michael Soyka

Michael Soyka wrote:

> The following statement
>
> if not exist $@ $(DEL) $@
>
> on line 98 in testdir/Make_ming.mak has two errors:
>
> 1. it attempts to delete a non-existing file and
> 2. the path that is substituted for $@ includes unix/linux path separators.
>
> The attached patch fixes both.

Thanks, I'll include it.

--
There are only two hard things in programming: Cache invalidation,
naming things and off-by-one errors.

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages