Skip to content

Conversation

@nixel2007
Copy link
Member

No description provided.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Подозреваю, что вместо "" нужно """" (не 2, а 4 кавычки)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хм, действительно. Странно, что у меня отработало без ошибок в таком варианте.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Видимо, потому что Каталог является последним в командной строке. Если бы за ним были ещё ключи или параметры, то они присоединились бы к каталогу. Кстати, в Windows есть ещё один подводный камень с обрамлением пути кавычками. Если путь заканчивается бэк-слешем , то после обрамления пути кавычками получается последовательность " , которая воспринимается командным интерпретатором как кавычка, являющаяся частью параметра. Т.е. опять же, если за таким путем будут идти другие параметры или ключи, они будут присоединены к этому пути как один большой параметр. Так что, по хорошему, ещё бы проверять, что Каталог не заканчивается на бэк-слеш.

artbear added a commit that referenced this pull request Nov 13, 2015
Исправлена ошибка добавления в индекс git файлов с пробелом в пути
@artbear artbear merged commit e818a14 into xDrivenDevelopment:develop Nov 13, 2015
@nixel2007 nixel2007 deleted the patch-1 branch November 13, 2015 06:45
@artbear
Copy link
Member

artbear commented Nov 13, 2015

@awa15

Видимо, потому что Каталог является последним в командной строке. Если бы за ним были ещё ключи или параметры, то они присоединились бы к каталогу. Кстати, в Windows есть ещё один подводный камень с обрамлением пути кавычками. Если путь заканчивается бэк-слешем , то после обрамления пути кавычками получается последовательность " , которая воспринимается командным интерпретатором как кавычка, являющаяся частью параметра. Т.е. опять же, если за таким путем будут идти другие параметры или ключи, они будут присоединены к этому пути как один большой параметр. Так что, по хорошему, ещё бы проверять, что Каталог не заканчивается на бэк-слеш.

Ага, очень верное замечание насчет последнего слеша
@nixel2007 Добавишь проверку на последний слеш и его удаление перед добавлением кавычек?

@nixel2007
Copy link
Member Author

@artbear
да, сделаю.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants