<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>57232</bug_id>
          
          <creation_ts>2025-12-15 14:57:09 +0300</creation_ts>
          <short_desc>Хардкод путей в cmake не позволяющий собрать во flatpak.</short_desc>
          <delta_ts>2025-12-30 14:10:04 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>surguch</component>
          <version>unstable</version>
          <rep_platform>x86_64</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P5</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Radiolamp">anthony_osipov</reporter>
          <assigned_to name="proskur@altlinux.org">proskur</assigned_to>
          <cc>anthony_osipov</cc>
    
    <cc>antohami</cc>
    
    <cc>krf10</cc>
    
    <cc>proskur</cc>
    
    <cc>proskurinov</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>279060</commentid>
    <comment_count>0</comment_count>
    <who name="Radiolamp">anthony_osipov</who>
    <bug_when>2025-12-15 14:57:09 +0300</bug_when>
    <thetext>235 install(FILES surguch.metainfo.xml DESTINATION ${SHARE_INSTALL_PREFIX}/metainfo PERMISSIONS WORLD_READ)
 236 
 237 IF (NOT DEFINED APP_ICON_DIR)
 238     SET(APP_ICON_DIR /usr/share/icons/hicolor/scalable/apps/)
 239 ENDIF ()
 240 IF (NOT DEFINED APP_ICON_PNG_DIR)
 241     SET(APP_ICON_PNG_DIR /usr/share/pixmaps/)
 242 ENDIF ()
Исправте пожалуйста это и другие места с хардкодолм</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279061</commentid>
    <comment_count>1</comment_count>
    <who name="Антон Мидюков">antohami</who>
    <bug_when>2025-12-15 15:01:57 +0300</bug_when>
    <thetext>То есть всего-то заменить /usr/share на ${SHARE_INSTALL_PREFIX}</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279065</commentid>
    <comment_count>2</comment_count>
    <who name="proskurinov@basealt.ru">proskurinov</who>
    <bug_when>2025-12-15 16:07:17 +0300</bug_when>
    <thetext>Все что захардкожено используется для локальной сборки из гита 
APP_ICON_DIR
APP_ICON_PNG_DIR 
можно указать из .spec файла, как и SHARE_INSTALL_PREFIX</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279066</commentid>
    <comment_count>3</comment_count>
    <who name="proskurinov@basealt.ru">proskurinov</who>
    <bug_when>2025-12-15 16:10:44 +0300</bug_when>
    <thetext>SHARE_INSTALL_PREFIX точно такая же hardcoded переменная, если другое не указано в spec файле.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279068</commentid>
    <comment_count>4</comment_count>
    <who name="proskurinov@basealt.ru">proskurinov</who>
    <bug_when>2025-12-15 16:17:45 +0300</bug_when>
    <thetext>сама семантика IF (NOT DEFINED) и Harcoded идут вразрез по смыслу.
Harcoded - это по определению что-то, прибитое гвоздями.
IF (NOT DEFINED) - это, скорее, значение по умолчанию. Которое, при желании, можно поменять на любое другое.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279088</commentid>
    <comment_count>5</comment_count>
    <who name="proskurinov@basealt.ru">proskurinov</who>
    <bug_when>2025-12-15 21:14:13 +0300</bug_when>
    <thetext>*** Bug 57231 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279679</commentid>
    <comment_count>6</comment_count>
    <who name="Антон Мидюков">antohami</who>
    <bug_when>2025-12-26 22:13:59 +0300</bug_when>
    <thetext>(Ответ для proskurinov@basealt.ru на комментарий #3)
&gt; SHARE_INSTALL_PREFIX точно такая же hardcoded переменная, если другое не
&gt; указано в spec файле.

Непонятно, зачем вам вот так определять APP_ICON_DIR и APP_ICON_PNG_DIR? SHARE_INSTALL_PREFIX, который задаётся в спеке в макросе %cmake чем вас не устраивает? При локальной сборке вам по какой-то причине нужно использовать иконки от установленного в систему? Но тогда вы могли бы задать эти переменные при локальной сборке.

Но хорошо, если вам это ломает ваш рабочий процесс, задайте в спеке явно эти переменные:
-%cmake -DCMAKE_BUILD_TYPE=Release -G Ninja
+%cmake -DCMAKE_BUILD_TYPE=Release -G Ninja \
+	-DAPP_ICON_DIR=%_iconsdir/hicolor/scalable/apps/ \
+	-DAPP_ICON_PNG_DIR=%_pixmapsdir/

Это сделать ещё более несложно.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279728</commentid>
    <comment_count>7</comment_count>
    <who name="proskurinov@basealt.ru">proskurinov</who>
    <bug_when>2025-12-29 09:21:57 +0300</bug_when>
    <thetext>Ну собственно я с вами полностью согласен, всё это можно и нужно указывать в .spec ,
переменные были нужны для каких то pipeline-ов на каком-то из этапов проекта.

Не вижу причин, почему бы они сейчас могли мешать, так как по факту - это не хардкод.
Сейчас вникать в необходимость/возможность их удаления не вижу смысла, потому что придется 
тестировать все пайплайны для kde/gnome/xfce/mate, выглядит, как пустая трата времени. 

Вы правы, их нужно явно указать в spec файле, вот здесь наша недоработка, видимо просто забыли/недосмотрели.

Собственно, я же сразу такое решение и предлагал - указать в спеке.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279839</commentid>
    <comment_count>8</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2025-12-30 14:10:04 +0300</bug_when>
    <thetext>surguch-0.4.3-alt2 -&gt; sisyphus:

Tue Dec 30 2025 Oleg Proskurin &lt;proskur@altlinux&gt; 0.4.3-alt2
- Fix major mistakes in the .spec file (closes #57232).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>