While the basics are clear, I found something that seems to be an
unintended mistake:

From test build log

...
4637 -- Installing: 
/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/bin/union-ruleinspector
4638 -- Set non-toolchain portion of runtime path of 
"/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/bin/union-ruleinspector"
 to ""
4639 -- Installing: 
/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/lib/x86_64-linux-gnu/cmake/Union/UnionTargets.cmake
4640 -- Installing: 
/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/lib/x86_64-linux-gnu/cmake/Union/UnionTargets-none.cmake
4641 -- Installing: 
/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/lib/x86_64-linux-gnu/cmake/Union/UnionConfig.cmake
4642 -- Installing: 
/build/reproducible-path/union-6.7.4/debian/kde-style-union/usr/lib/x86_64-linux-gnu/cmake/Union/UnionConfigVersion.cmake
...

# ok it installs its cmake files
# Then from d/rules

4646 # remove development files                                                 
      
4647 rm -f debian/kde-style-union/usr/lib/*/libUnion.so                         
      
4648 rm -rf debian/union/usr/include/                                           
      
4649 rm -rf debian/union/usr/lib/*/cmake/ 

# Ok, so you do not want them
# But that path is not correct.

# It is not debian/union/usr/lib/*/cmake/ but 
debian/kde-style-union/usr/lib/*/cmake
# Due to that the final .deb file has

5395 drwxr-xr-x root/root         0 2026-08-20 13:29 
./usr/lib/x86_64-linux-gnu/cmake/Union/
5396 -rw-r--r-- root/root      1493 2026-08-20 13:29 
./usr/lib/x86_64-linux-gnu/cmake/Union/UnionConfig.cmake
5397 -rw-r--r-- root/root      1861 2026-08-20 13:29 
./usr/lib/x86_64-linux-gnu/cmake/Union/UnionConfigVersion.cmake
5398 -rw-r--r-- root/root       856 2026-08-20 13:29 
./usr/lib/x86_64-linux-gnu/cmake/Union/UnionTargets-none.cmake
5399 -rw-r--r-- root/root      5722 2026-08-20 13:29 
./usr/lib/x86_64-linux-gnu/cmake/Union/UnionTargets.cmake

# Same for the include file AFAICS

I checked other kde-style packages and they do not have that, so it
seems to be a genuine mistake.

This is not blocking this to be acceptable, so I'll accept it from NEW.
But you may want to fix this up in a follow up upload?


P.S. shedding a tear for -DBUILD_TESTING=OFF, just out of principle :-)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2163760

Title:
  [needs packaging] Union - New qt6 style for KDE

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/2163760/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to