Bug 29671 - Переместить в %_includedir/amd/
Summary: Переместить в %_includedir/amd/
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: libxvba-devel (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: led
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-27 16:12 MSK by Sergey V Turchin
Modified: 2015-01-12 18:35 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey V Turchin 2013-12-27 16:12:20 MSK
Просьба переместить amdxvba.h в %_includedir/amd/ , т.к. его ищут там.
Comment 1 led 2013-12-27 17:10:00 MSK
(В ответ на комментарий №0)
> Просьба переместить amdxvba.h в %_includedir/amd/ , т.к. его ищут там.

Кто?
Comment 2 Sergey V Turchin 2013-12-27 17:18:15 MSK
avidemux и еще кто-то 
http://forum.xbmc.org/showthread.php?tid=116996&pid=967607#pid967607
Comment 4 Sergey V Turchin 2013-12-27 17:28:52 MSK
Или проще засимлинкать, чтоб всем угодить.
avidemux патчить совсем не хочется. Там и так наколбашено.
Comment 5 led 2013-12-27 17:30:39 MSK
(В ответ на комментарий №2)
> avidemux и еще кто-то 
> http://forum.xbmc.org/showthread.php?tid=116996&pid=967607#pid967607

А другие ищут в include/
И в инструкции от разрабочика API сказано:
"Install the header file (`include/amdxvba.h') in a directory where the C compiler can find it, typically `/usr/local/include'."

Я не вижу причин не следовать этому.
Comment 6 led 2013-12-27 17:33:11 MSK
(В ответ на комментарий №4)
> Или проще засимлинкать, чтоб всем угодить.
> avidemux патчить совсем не хочется. Там и так наколбашено.

ИМХО патчить нужно то, что сломано, а не для того, чтобы угодить сломанному. В данном случае "сломано" то, что ищет (почему-то) в include/amd/.
Comment 7 Sergey V Turchin 2013-12-27 17:37:02 MSK
(В ответ на комментарий №6)
> ИМХО патчить нужно то, что сломано
1. IMHO сломано в libxvba-devel
2. Будете патчить?
Я запушил http://git.altlinux.org/people/zerg/packages/?p=avidemux-qt.git , посмотрите.
Comment 8 led 2013-12-27 17:45:38 MSK
(В ответ на комментарий №7)
> (В ответ на комментарий №6)
> > ИМХО патчить нужно то, что сломано
> 1. IMHO сломано в libxvba-devel

Я привел цитату. Упаковано так, как это преполагает AMD.

> 2. Будете патчить?

Нет, потому что это сломает сборку того, кто использует amdxvba.h так, как указано разрабочиком (в частности пакет xvba-video, для которого libxvba-devel изначально и предполагался).

> Я запушил http://git.altlinux.org/people/zerg/packages/?p=avidemux-qt.git ,
> посмотрите.

Посмотрел. У меня нет никаких возражений. И не может быть, раз всё работает:)
Comment 9 Sergey V Turchin 2013-12-27 18:00:03 MSK
(В ответ на комментарий №8)
> > 2. Будете патчить?
> Нет, потому что это сломает сборку того, кто использует amdxvba.h так
1. Я про avidemux :-)
2. Я предлажил засимлинкать -- ничего не сломает.

> Посмотрел. У меня нет никаких возражений.
> И не может быть, раз всё работает:)
Что вы смотрели, если ничего не работает?
Comment 10 Sergey V Turchin 2013-12-27 18:00:59 MSK
Достаточно создать %_includedir/amd/ и засимлинкать туда или обратно.
Comment 11 led 2013-12-27 18:28:32 MSK
(В ответ на комментарий №9)
> > Посмотрел. У меня нет никаких возражений.
> > И не может быть, раз всё работает:)
> Что вы смотрели, если ничего не работает?

grep -rlw 'amd/amdxvba\.h' | xargs sed -i 's|amd/\(amdxvba\.h\)|\1|g'
Comment 12 Sergey V Turchin 2013-12-27 18:39:22 MSK
(В ответ на комментарий №11)
> grep -rlw 'amd/amdxvba\.h' | xargs sed -i 's|amd/\(amdxvba\.h\)|\1|g'
Хорошо, но костыль больше. Почему бы не засимлинкать?
Comment 13 led 2013-12-27 18:45:42 MSK
(В ответ на комментарий №12)
> (В ответ на комментарий №11)
> > grep -rlw 'amd/amdxvba\.h' | xargs sed -i 's|amd/\(amdxvba\.h\)|\1|g'
> Хорошо, но костыль больше.

Что костыльного в том, чтобы выполнить эту команду и сделать "git commit..."?

> Почему бы не засимлинкать?

Потому что это как раз и будет костыль - для конкретного кривого апстрима.

Из того что я нашёл, только Ubuntu почему-то кладёт в include/amd/ - больше никто. Похоже, апстрим (или автор vaapi-патча) сидит на Ubuntu - вот и нагородил...
Comment 14 led 2013-12-27 18:52:36 MSK
(В ответ на комментарий №13)
> автор vaapi-патча

xvba-патча конечно же.
Comment 15 Sergey V Turchin 2013-12-27 18:55:42 MSK
(В ответ на комментарий №13)
> Что костыльного в том, чтобы выполнить эту команду и сделать "git
> commit..."?
У меня это не работает. Только если в конце секции %prep .
Comment 16 Sergey V Turchin 2013-12-27 18:59:24 MSK
Ну да ладно. Пусть так.
Comment 17 Sergey V Turchin 2013-12-27 19:04:02 MSK
(В ответ на комментарий №13)
> Что костыльного в том, чтобы выполнить эту команду и сделать
> "git commit..."?
Я вообще против такого. Потом фиг разберешь, где какой патч.
Comment 18 led 2013-12-27 19:16:55 MSK
(В ответ на комментарий №15)
> (В ответ на комментарий №13)
> > Что костыльного в том, чтобы выполнить эту команду и сделать "git
> > commit..."?
> У меня это не работает. Только если в конце секции %prep .

git clone git://git.altlinux.org/people/zerg/packages/avidemux-qt.git
cd avidemux-qt.git
grep -rlw 'amd/amdxvba\.h' avidemux | xargs sed -i 's|amd/\(amdxvba\.h\)|\1|g'
git commit -a -m bla-bla-bla
gear --commit --rpmbuild -v -- rpmbuild -bi --target=x86_64

- всё работает.
Comment 19 Sergey V Turchin 2014-01-09 16:43:53 MSK
(В ответ на комментарий №18)
> - всё работает.
Я ж говорю, _у_меня_ не работает. См. коментарий #17 .
Comment 20 Michael Shigorin 2015-01-12 18:35:17 MSK
(В ответ на комментарий №17)
> > Что костыльного в том, чтобы выполнить эту команду и сделать
> > "git commit..."?
> Я вообще против такого. Потом фиг разберешь, где какой патч.
Год спустя: подобные трансформации предпочитаю держать в %setup, чтоб хранилось в "интегральном" виде, а не как результат выполнения.  Разумеется, у такого подхода есть свой недостаток -- отваливается молча...