bem-archive / bem-tools

Toolkit to work with files based on BEM methodology
http://bem.info/tools/bem/
MIT License
395 stars 72 forks source link

Wrong bem make behavior on not-so-correct bemjson.js tech declaration #313

Closed arikon closed 9 years ago

scf2k commented 11 years ago

Там не тот тип ноды создается по какой-то причине. Подозреваю что баг появился после рефакторинга create-bemdecl.js-node.

arikon commented 11 years ago

Я покопал, проблема была потом, что в getTechs() уровня было написано так: { 'bemdecl.js': '' } вместо { 'bemdecl.js': 'bemdecl.js' }. При этом создавался файл .tech для страницы, удалялся .bemdecl.js при bem make -m clean. Такого не должно быть, надо это поведение чинить.

scf2k commented 11 years ago

Так ведь '' и должно быть для исходных файлов?

arikon commented 11 years ago

Сборка должна работать правильно не зависимо от того, как именно задекларирована технология.

scf2k commented 11 years ago

Как определить исходный это файл или нет? Удалять его или нет?

arikon commented 11 years ago

По наличию файла *.bemjson.js на диске. Эта логика же вроде есть в BundleNode?