Bug 24120 - MKI_PACK_RESULTS с значением data
Summary: MKI_PACK_RESULTS с значением data
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: mkimage (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-20 12:41 MSD by Andriy Stepanov (stanv)
Modified: 2010-10-05 23:46 MSD (History)
3 users (show)

See Also:


Attachments
Пример профиля на котором падает сборка (650 bytes, text/plain)
2010-09-20 12:41 MSD, Andriy Stepanov (stanv)
no flags Details
Лог по сборке профиля (573.32 KB, text/plain)
2010-09-20 12:42 MSD, Andriy Stepanov (stanv)
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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).