Skip to content
This repository has been archived by the owner on Jan 13, 2018. It is now read-only.

Incorrect handling of paths on windows #27

Closed
zemd opened this issue Dec 24, 2011 · 4 comments
Closed

Incorrect handling of paths on windows #27

zemd opened this issue Dec 24, 2011 · 4 comments
Labels

Comments

@zemd
Copy link

zemd commented Dec 24, 2011

Добрый день, первый раз сталкиваюсь с nodejs поэтому могу что-то не правильно делать.

Поставил node.exe версии 0.6.6 на windows 7 x64, пользуюсь ryppi пакет менеджером. Тестовые проекты для nodejs работают.

Установил bem. Но при обращении к утилите - резальтатов никаких не даёт. Опытным путём (console.log) - выяснил что методы .act() не вызываются никогда.

к примеру в файле bem/lib/commands/create/level.js после строчки "module.exports = function() {" -- console.log(); выполняется, а далее по коду в методе act() -- нет. Запускал с командой bem create level blocks.

Так же не отвечают на запросы все другие команды - даже bem --help ничего не выводит.

@ghost ghost assigned gfranco Dec 24, 2011
@zemd
Copy link
Author

zemd commented Dec 25, 2011

еще можно дополнить - поставл npm 1.1.0-beta-7 (думал может его созданные cmd файлы мне помогут ) - но не помогло. запустил git bash - аналогичная ситуация - bem запускается, но ничего не делает.

@zemd
Copy link
Author

zemd commented Dec 25, 2011

поразбирался еще немного и нашел ошибку - в функции exports.findLevel файла bem/lib/util.js

логика такая:

  1. exports.findLevel (E:\dev\nodejs) -- вызывается с параметром process.cwd()
  2. функция isLevel возвращает false т.к. не находит .bem/level.js
  3. exports.findLevel (E:\dev) -- PATH.dirname(E:\dev\nodejs) = E:\dev
  4. тоже повторяется что и в пункте 2
  5. exports.findLevel (E:) -- PATH.dirname(E:\dev) = E:\
  6. зацикливается т.к. E:\ никогда не будет равен '/' -- (if (path == '/') return startPath;)

заменил пока / на E:\ и всё работает. выводится и версия bem-а и создаются каталоги.

@tttptd
Copy link

tttptd commented Dec 29, 2011

Спасибо, мне помогло.
Вроде бы только строка 138.

@arikon
Copy link
Contributor

arikon commented Jan 17, 2012

Will be in next release

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants