Bug 34100 - non-identical noarch PNG image, 8-bit/color RGBA - hack around
Summary: non-identical noarch PNG image, 8-bit/color RGBA - hack around
Status: CLOSED FIXED
Alias: None
Product: Infrastructure
Classification: Infrastructure
Component: girar (show other bugs)
Version: unspecified
Hardware: all Linux
: P3 enhancement
Assignee: Dmitry V. Levin
QA Contact: Andrey Cherepanov
URL: http://git.altlinux.org/people/viy/pa...
Keywords:
Depends on: 34101
Blocks:
  Show dependency tree
 
Reported: 2017-10-30 21:36 MSK by viy
Modified: 2018-06-10 13:42 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description viy 2017-10-30 21:36:09 MSK
если doxygen никто не смог/не нашел нужным починить, надо чинить girar.
Сейчас там по сути издевательство с проверкой non-identical noarch packages.

@@ -106,3 +106,3 @@
-/usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleFinder.png    100644  PNG image, 8-bit/color RGB, non-interlaced
+/usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleFinder.png    100644  PNG image, 8-bit/color RGBA, non-interlaced
error (#100): non-identical noarch packages
2017-Oct-26 19:06:28 :: noarch check FAILED

Есть готовый патч.

http://git.altlinux.org/people/viy/packages/?p=girar.git;a=commit;h=288e3d514af41a39640946e7ee530146ba2bd66e
--- a/gb/gb-task-check-noarch-i
+++ b/gb/gb-task-check-noarch-i
@@ -63,6 +63,7 @@ dump_rpmfile()
                { echo >&2 "${1##*/}: unpack failed"; return 1; }
        sed -i 's/, last modified: [^,]\+//' $tmpdir/types
        sed -i 's/\(PNG image\), [1-9][^,]\+/\1/' $tmpdir/types
+       sed -i '/PNG image/s,8-bit/color RGBA,8-bit/color RGB,' $tmpdir/types
        sed -i 's/\(GIF image data, version 89a\), [1-9][0-9]* x [1-9][0-9]*/\1/' $tmpdir/types
        sed -i 's/\(TeX DVI file \)(TeX output .*/\1/' $tmpdir/types
        sed -i 's/\(DBase 3 data file \)([0-9]* records)/\1/' $tmpdir/types
Comment 1 Dmitry V. Levin 2017-10-30 23:21:26 MSK
(In reply to comment #0)
> если doxygen никто не смог/не нашел нужным починить,

Видимо, не doxygen, а convert.
Comment 2 Dmitry V. Levin 2017-10-30 23:25:25 MSK
Для иллюстрации: #25711

http://git.altlinux.org/gears/t/..git?p=tango-icon-theme.git;a=commitdiff;h=878c952
Comment 3 Dmitry V. Levin 2017-10-31 00:02:42 MSK
Впрочем, это может быть и dot из пакета graphviz.
Comment 4 Gleb F-Malinovskiy 2017-10-31 00:21:31 MSK
(In reply to comment #2)
> Для иллюстрации: #25711
> 
> http://git.altlinux.org/gears/t/..git?p=tango-icon-theme.git;a=commitdiff;h=878c952

Там это выглядит (без патча) вот так, если что:
-/usr/share/icons/Tango/24x24/status/weather-storm.png:                          PNG image, 24 x 24, 8-bit colormap, non-interlaced
+/usr/share/icons/Tango/24x24/status/weather-storm.png:                          PNG image, 24 x 24, 8-bit/color RGBA, non-interlaced
Comment 5 viy 2017-10-31 00:50:54 MSK
соединяя оба случая в один, предлагаю улучшенный патч

diff --git a/gb/gb-task-check-noarch-i b/gb/gb-task-check-noarch-i
index be37ef0..3e97d91 100755
--- a/gb/gb-task-check-noarch-i
+++ b/gb/gb-task-check-noarch-i
@@ -63,6 +63,7 @@ dump_rpmfile()
                { echo >&2 "${1##*/}: unpack failed"; return 1; }
        sed -i 's/, last modified: [^,]\+//' $tmpdir/types
        sed -i 's/\(PNG image\), [1-9][^,]\+/\1/' $tmpdir/types
+       sed -i '/PNG image/s@, 8-bit\( /color RGBA?\| colormap\)@@' $tmpdir/type
        sed -i 's/\(GIF image data, version 89a\), [1-9][0-9]* x [1-9][0-9]*/\1/
        sed -i 's/\(TeX DVI file \)(TeX output .*/\1/' $tmpdir/types
        sed -i 's/\(DBase 3 data file \)([0-9]* records)/\1/' $tmpdir/types
Comment 6 viy 2017-10-31 00:52:44 MSK
точнее,
http://git.altlinux.org/people/viy/packages/?p=girar.git;a=commit;h=67e3025203453b61580ef9e8e166b825d4ac9c81

sed -i '/PNG image/s@, 8-bit\(/color RGBA?\| colormap\)@@' $tmpdir/types
Comment 7 Dmitry V. Levin 2017-10-31 01:05:58 MSK
Зачем вы таск удалили?  Верните, пожалуйста, поскорее.
Comment 8 viy 2017-10-31 01:13:20 MSK
(В ответ на комментарий №7)
> Зачем вы таск удалили?  Верните, пожалуйста, поскорее.

перезалил новый патч
Comment 9 Dmitry V. Levin 2017-10-31 01:18:59 MSK
(In reply to comment #8)
> (В ответ на комментарий №7)
> > Зачем вы таск удалили?  Верните, пожалуйста, поскорее.
> 
> перезалил новый патч

Пакет, на котором это воспроизводится, верните, пожалуйста, поскорее.
Comment 10 viy 2017-10-31 01:24:14 MSK
(В ответ на комментарий №9)
> Пакет, на котором это воспроизводится, верните, пожалуйста, поскорее.

#192741 AWAITING #1 sisyphus srpm=jboss-modules-1.5.2-alt2_2jpp8.src.rpm
Comment 11 viy 2017-10-31 01:56:49 MSK
вот тест таск

2017-Oct-30 22:37:16 :: task #192741 for sisyphus started by viy:
#100 build jboss-modules-1.5.2-alt2_2jpp8.src.rpm
2017-Oct-30 22:37:16 :: waiting for a shared lock on sisyphus
2017-Oct-30 22:52:38 :: acquired a shared lock on sisyphus
2017-Oct-30 22:52:38 :: [i586] #100 jboss-modules-1.5.2-alt2_2jpp8.src.rpm: build start
2017-Oct-30 22:52:38 :: [x86_64] #100 jboss-modules-1.5.2-alt2_2jpp8.src.rpm: build start
2017-Oct-30 22:54:41 :: [x86_64] #100 jboss-modules-1.5.2-alt2_2jpp8.src.rpm: build OK
2017-Oct-30 22:54:45 :: [i586] #100 jboss-modules-1.5.2-alt2_2jpp8.src.rpm: build OK
2017-Oct-30 22:55:11 :: build check OK
--- jboss-modules-javadoc-1.5.2-alt2_2jpp8.noarch.rpm.i586      2017-10-30 22:55:12.904329043 +0000
+++ jboss-modules-javadoc-1.5.2-alt2_2jpp8.noarch.rpm.x86_64    2017-10-30 22:55:13.501324984 +0000
@@ -106,3 +106,3 @@
 /usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleFinder.html   100644  HTML document text
-/usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleFinder.png    100644  PNG image, 8-bit/color RGB, non-interlaced
+/usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleFinder.png    100644  PNG image, 8-bit/color RGBA, non-interlaced
 /usr/share/javadoc/jboss-modules/org/jboss/modules/ModuleIdentifier.html       100644  HTML document text
error (#100): non-identical noarch packages
2017-Oct-30 22:55:13 :: noarch check FAILED
2017-Oct-30 22:55:13 :: task #192741 for sisyphus FAILED
Comment 12 Dmitry V. Levin 2017-10-31 06:10:00 MSK
Они даже визуально сильно отличаются.
Это мерзкое поведение dot'а нужно прекратить.
Comment 13 Michael Shigorin 2017-10-31 14:23:52 MSK
(В ответ на комментарий №12)
> Они даже визуально сильно отличаются.
> Это мерзкое поведение dot'а нужно прекратить.
Патчи, увы, приветствуются.
Comment 14 viy 2017-11-01 09:46:11 MSK
судя по
[#192741] DONE (try 2) srpm=jboss-modules-1.5.2-alt2_2jpp8.src.rpm
fixed?