Bug 24120

Summary: MKI_PACK_RESULTS с значением data
Product: Sisyphus Reporter: Andriy Stepanov (stanv) <stanv>
Component: mkimageAssignee: Alexey Gladkov <legion>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: glebfm, legion, mike
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
Пример профиля на котором падает сборка
none
Лог по сборке профиля none

Description Andriy Stepanov (stanv) 2010-09-20 12:41:52 MSD
Created attachment 4551 [details]
Пример профиля на котором падает сборка

В mkimage-0.1.5-alt1 не правильно отрабатывает цель pack-image
для типа упаковки data:

#  <MKI_PACKTYPE>[:<MKI_OUTNAME>[:[!]<SUBDIR>,...]]
#   * Kernel image, /boot directory
MKI_PACK_RESULTS = data:boot:/boot
Comment 1 Andriy Stepanov (stanv) 2010-09-20 12:42:36 MSD
Created attachment 4552 [details]
Лог по сборке профиля
Comment 2 Michael Shigorin 2010-09-21 11:29:34 MSD
(In reply to comment #0)
> MKI_PACK_RESULTS = data:boot:/boot
А это что за синтаксис?

http://git.altlinux.org/people/legion/packages/mkimage.git?p=mkimage.git;a=blob;f=doc/README.ru;hb=HEAD#l287

<PACKTYPE>:<OUTNAME>[:<SUBDIR>] [<PACKTYPE1>:<OUTNAME1>[:<SUBDIR1>] ...]
Comment 3 Andriy Stepanov (stanv) 2010-09-21 17:15:26 MSD
Нормальный синтаксис.
Он еще не документирован:
http://git.altlinux.org/people/legion/packages/mkimage.git?p=mkimage.git;a=blob;f=tools/mki-pack-results;h=b864b8a7e7bbbab55797d008d6c2f57254f5e824;hb=HEAD
Comment 4 Andriy Stepanov (stanv) 2010-10-05 14:18:28 MSD
> grep -rn 'MKI_IMAGESUBDIRS' .           

./mki-pack-results:38:  MKI_IMAGESUBDIRS=
./mki-pack-results:54:                  MKI_IMAGESUBDIRS="$a" ||
./mki-pack-results:58:  MKI_IMAGESUBDIRS="${MKI_IMAGESUBDIRS# }"

> grep -rn 'MKI_IMAGESUBDIR[^S]' .

./mki-copy-isolinux:18:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-data:14:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-copy-yaboot:16:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-yaboot:22:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-tar:21:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-custom:21:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-isoboot:21:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-squash:25:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-results:63:  local MKI_PACKMOD= MKI_PACKTYPE= MKI_OUTNAME= MKI_IMAGESUBDIR= MKI_EXCLUDE=
./mki-pack-isodata:21:imgsubdir="${MKI_IMAGESUBDIR:-}"
./mki-pack-cpio:21:imgsubdir="${MKI_IMAGESUBDIR:-}"

Исправление в:
http://git.altlinux.org/people/stanv/packages/mkimage.git
Comment 5 Alexey Gladkov 2010-10-05 14:58:51 MSD
Вы поменяли MKI_PACK_RESULTS и добавили в него множественные SUBDIR. Это должно делаться не так, а внутри каждого mki-pack-*.

Опечатка с MKI_IMAGESUBDIR{S} исправлена тут:

http://git.altlinux.org/people/legion/packages/mkimage.git?p=mkimage.git;a=commitdiff;h=5a89c1a66d9d56e7f5e7b21b87af955b2fe86dfe

пожалуйста проверьте.
Comment 6 Andriy Stepanov (stanv) 2010-10-05 15:32:38 MSD
Проверил.
Цель
MKI_PACK_RESULTS = data:boot:/boot
теперь отрабатывает нормально.
Выложи пожалуйста в сизиф.
Comment 7 Repository Robot 2010-10-05 23:46:45 MSD
mkimage-0.1.6-alt1 -> sisyphus:

* Tue Oct 05 2010 Alexey Gladkov <legion@altlinux> 0.1.6-alt1
- mki-pack-results: Fix MKI_IMAGESUBDIR (ALT#24120).
- Add example3 (thx Michael Shigorin).