Summary: | Ошибка обновления | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | Dmytro <d20052005> |
Component: | liblua5.1 | Assignee: | viy <viy> |
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
Severity: | normal | ||
Priority: | P3 | CC: | evg, viy |
Version: | unstable | ||
Hardware: | all | ||
OS: | Linux |
Description
Dmytro
2017-02-11 23:20:38 MSK
task #178041: added #100: build tag "5.1.5-alt8" from /people/viy/packages/lua5.1.git попробуйте, пожалуйста. Еще не пробовал, но посмотрел новый spec. Есть замечание. %pre -n lib%{name} # ----------- begin update from old lua5 to lua5.1 ---- if [ -L %_libdir/lua/5.1 ]; then echo "removing lua5.1-alt-compat symlink..." rm -f %_libdir/lua/5.1 if [ -d %_libdir/lua5 ] && [ ! -e %_libdir/lua/5.1 ]; then mkdir -p %_libdir/lua mv %_libdir/lua5 %_libdir/lua/5.1 fi fi # ----------- end update from old lua5 to lua5.1 ---- У Вас сначала идет проверка, есть ли ссылка %_libdir/lua/5.1, затем, если есть, ее удаление, и затем во вложенном условии снова проверка, есть ли файл/каталог %_libdir/lua/5.1. В случае, если первое условие не выполнится, то до > if [ -d %_libdir/lua5 ] && [ ! -e %_libdir/lua/5.1 ]; then дело не дойдет. А если > if [ -L %_libdir/lua/5.1 ]; будет истинно, то после > rm -f %_libdir/lua/5.1 второе тоже станет истинно. Предлагаю вынести вложенный if наружу, т.е. записать вот в таком виде: %pre -n lib%{name} # ----------- begin update from old lua5 to lua5.1 ---- if [ -L %_libdir/lua/5.1 ]; then echo "removing lua5.1-alt-compat symlink..." rm -f %_libdir/lua/5.1 fi if [ -d %_libdir/lua5 ] && [ ! -e %_libdir/lua/5.1 ]; then mkdir -p %_libdir/lua mv %_libdir/lua5 %_libdir/lua/5.1 fi # ----------- end update from old lua5 to lua5.1 ---- (In reply to comment #3) > Еще не пробовал, но посмотрел новый spec. Есть замечание. > У Вас сначала идет проверка, > if [ -L %_libdir/lua/5.1 ]; then > if [ -d %_libdir/lua5 ] && [ ! -e %_libdir/lua/5.1 ]; then [...] Да, избыточное. Поправлю, если будет новый релиз. > Предлагаю вынести вложенный if наружу, т.е. записать вот в таком виде: Так не хочу, хочу только когда был найден симлинк. в остальных случаях лучше не трогать. (In reply to comment #4) > (In reply to comment #3) > > Предлагаю вынести вложенный if наружу, т.е. записать вот в таком виде: > > Так не хочу, хочу только когда был найден симлинк. > в остальных случаях лучше не трогать. Если файлы из lua5 в lua5.1 переносить не обязательно, то Вы правы. (In reply to comment #5) > (In reply to comment #4) > > (In reply to comment #3) > > > > Предлагаю вынести вложенный if наружу, т.е. записать вот в таком виде: > > > > Так не хочу, хочу только когда был найден симлинк. > > в остальных случаях лучше не трогать. > > Если файлы из lua5 в lua5.1 переносить не обязательно, то Вы правы. да. Все пакеты, которые там держали что-либо, будут пересобраны. Проверил. Обновление с p8 до Сизифа проходит корректно. Меняю на "FIXED" |