8089 2005-09-29 14:54:30 +0400 Make rpmbuild fail on unknown macros 2005-10-26 23:39:49 +0400 1 1 4 Development Sisyphus rpm-build unstable all Linux CLOSED FIXED http://lists.altlinux.ru/pipermail/devel/2005-October/025229.html P2 enhancement --- 1 raorn placeholder arseny glebfm imz ktirf ldv mike placeholder sbolshakov vt qa-sisyphus oldest_to_newest 31150 0 raorn 2005-09-29 14:54:30 +0400 Unknown macros should generate error when parsing spec. 31151 1 1151 raorn 2005-09-29 14:55:28 +0400 Created attachment 1151 rpm-4_0-4.0.4-alt-fail-on-unknown-macros.patch patch against rpm-4.0.4-alt48 31155 2 ldv 2005-09-29 17:22:49 +0400 Вероятно, этот патч неполный, там, кажется, есть ещё один случай, но суть не в этом. Я так понял, никто не возражает и последствий не боится? 31159 3 legion 2005-09-29 17:41:33 +0400 Я думаю ничего страшного не случится ... отваляться пакеты которые сейчас работают неправильно... если они вообще работают. 31162 4 raorn 2005-09-29 17:48:58 +0400 Поступило предположение, что сломается sandman в некоторых местах из-за отсутствия /etc/rpm/macros.d/*... 31165 5 raorn 2005-09-29 18:26:44 +0400 (In reply to comment #2) > Вероятно, этот патч неполный, там, кажется, есть ещё один случай, но суть не в > этом. Там ещё чуть выше есть, сразу после case(){}... Обновить патч? 31166 6 raorn 2005-09-29 18:31:48 +0400 (In reply to comment #4) > Поступило предположение, что сломается sandman в некоторых местах из-за > отсутствия /etc/rpm/macros.d/*... sandman желает знать Name-Version-Release, все Source/Patch и BuildRequires. Если какие-то макросы влияли на содержимое этих тегов раньше, значит они все раскрывались. Всё остальное можно скопировать после минимально необходимой информации, думаю таких случаев будет немного и решаться это будет мантейнерами этих пакетов... Мне кажется что конечный результат в какой-то степени оправдывает это количество геммороя. 31167 7 sbolshakov 2005-09-29 18:54:45 +0400 sandman рассчитывает на то, что rpm -bE выдаст значения некоторых тэгов (см. предыдущий комментарий), даже если в процессе препроцессинга есть ошибки. Иными словами, вывод команды не будет прекращён на первой нераскрывшейся конструкции. Если предлагаемый патч изменяет это поведение, то для sandman это закончится плохо. 31168 8 ldv 2005-09-29 19:33:46 +0400 Нет, предлагаемый патч работать не будет совершенно: если сделать нечто подобное, то все тэги пойдут лесом. 31170 9 ldv 2005-09-29 21:15:17 +0400 Чтобы было понятно: error: Macro %description not found 31174 10 raorn 2005-09-29 22:54:34 +0400 (In reply to comment #9) > Чтобы было понятно: > error: Macro %description not found Ох, как всё запущено... Я сходу придумал три варианта обхода этой проблемы и все три кривые до безобразия... 31188 11 raorn 2005-09-30 11:09:28 +0400 > Ох, как всё запущено... Я сходу придумал три варианта обхода этой проблемы и > все три кривые до безобразия... 1. subst 's/^%\(package\|description\|...\)/%%\1/' *.spec 2. где-то в macros: %global package %%package %global description %%description ... 3. как-то проверять имена секций в expandMacro(), например по build/parseSpec.c:partList[] Учитывая comment #7 нужен режим, где нераскрытые макросы не будут генерить ошибку, тоже слабо представляю как такое реализовать... P.S. Наверно пока RESOLVED/LATER... 31471 12 ldv 2005-10-06 23:19:15 +0400 Как насчёт --define '_allow_undefined_macros 1', с помощью которого можно будет обрабатывать неопределённые макросы? 31518 13 ldv 2005-10-07 18:33:51 +0400 Implemented in 4.0.4-alt49. 1151 2005-09-29 14:55:28 +0400 2005-09-29 14:55:28 +0400 rpm-4_0-4.0.4-alt-fail-on-unknown-macros.patch rpm-4_0-4.0.4-alt-fail-on-unknown-macros.patch text/plain 804 raorn ZGlmZiAtTmF1cnAgcnBtLTRfMC00LjAuNC1vcmlnL3JwbWlvL21hY3JvLmMgcnBtLTRfMC00LjAu NC9ycG1pby9tYWNyby5jCi0tLSBycG0tNF8wLTQuMC40LW9yaWcvcnBtaW8vbWFjcm8uYwkyMDA1 LTA2LTI5IDIyOjA0OjQ2ICswNDAwCisrKyBycG0tNF8wLTQuMC40L3JwbWlvL21hY3JvLmMJMjAw NS0wOS0yOSAxNDozNjozMCArMDQwMApAQCAtMTQzNSwyMyArMTQzNSwxMCBAQCBleHBhbmRNYWNy byhNYWNyb0J1ZiBtYikKIAkJY29udGludWU7CiAJfQogCQotCWlmIChtZSA9PSBOVUxMKSB7CS8q IGxlYXZlIHVua25vd24gJS4uLiBhcyBpcyAqLwotI2lmbmRlZiBIQUNLCi0jaWYgREVBRAotCQkv KiBYWFggaGFjayB0byBza2lwIG92ZXIgZW1wdHkgYXJnIGxpc3QgKi8KLQkJaWYgKGZuID09IDEg JiYgKmYgPT0gJyonKSB7Ci0JCQlzID0gc2U7Ci0JCQljb250aW51ZTsKLQkJfQotI2VuZGlmCi0J CS8qIFhYWCBoYWNrIHRvIHBlcm1pdCBub24tb3ZlcmxvYWRlZCAlZm9vIHRvIGJlIHBhc3NlZCAq LwotCQljID0gJyUnOwkvKiBYWFggb25seSBuZWVkIHRvIHNhdmUgJSAqLwotCQlTQVZFQ0hBUiht YiwgYyk7Ci0jZWxzZQorCWlmIChtZSA9PSBOVUxMKSB7CS8qIGJhaWwgb24gdW5rbm93biAlLi4u ICovCiAJCXJwbUVycm9yKFJQTUVSUl9CQURTUEVDLAotCQkJXygiTWFjcm8gJSUlLipzIG5vdCBm b3VuZCwgc2tpcHBpbmdcbiIpLCBmbiwgZik7Ci0JCXMgPSBzZTsKLSNlbmRpZgorCQkJXygiTWFj cm8gJSUlLipzIG5vdCBmb3VuZFxuIiksIGZuLCBmKTsKKwkJcmMgPSAxOwogCQljb250aW51ZTsK IAl9CiAK