Removed unused scripts and libtqtqui configuration #39

Closed
MicheleC wants to merge 1 commits from drop/unused-code into master
Owner

libtqtqui.so was not provided by any package.

libtqtqui.so was not provided by any package.
MicheleC added this to the R14.2.0 release milestone 3 months ago
MicheleC added 1 commit 3 months ago
fc0919ae69
Removed unused scripts and libtqtqui configuration
MicheleC requested review from Core 3 months ago
MicheleC requested review from Owners 3 months ago
SlavekB requested changes 3 months ago
SlavekB left a comment
Owner

Part concerning tqtqui.pc now incorrect. This requires a solution in a separate PR.

Part concerning `tqtqui.pc` now incorrect. This requires a solution in a separate PR.
# tqt.pc and tqtqui.pc definitions
# tqt.pc definitions
Owner

File tqtqui.pc file is still used.

File `tqtqui.pc` file is still used.
configure_file( tqtqui.pc.cmake tqtqui.pc @ONLY )
install( FILES ${CMAKE_CURRENT_BINARY_DIR}/tqt.pc DESTINATION ${PKGCONFIG_INSTALL_DIR} )
install( FILES ${CMAKE_CURRENT_BINARY_DIR}/tqtqui.pc DESTINATION ${PKGCONFIG_INSTALL_DIR} )
Owner

The file needs to be installed.

The file needs to be installed.
endforeach( )
configure_file( tqt.pc.cmake tqt.pc @ONLY )
configure_file( tqtqui.pc.cmake tqtqui.pc @ONLY )
Owner

The file needs to be generated.

The file needs to be generated.
Name: TQtQUI
Description: Interface and abstraction library for Qt and Trinity (QUI library)
Version: @TQT_VERSION@
Libs: -L${libdir} -ltqt -ltqui -L@TQT_LIBRARY_DIR@ @PC_TQT_LIBRARIES@
Owner

The pkg-config file is called tqtqui, but the library itself is in TQt3 and is called libtqui.so and it seems to be still used. See comments in TDE/tde-cmake#99. Removing this file therefore seems to be incorrect or, respectively, when merging tqtinterface to TQt3, this file will need to be generated in TQt3. Perhaps with the name tqui.pc, which corresponds to the name of the library.

The pkg-config file is called `tqtqui`, but the library itself is in TQt3 and is called `libtqui.so` and it seems to be still used. See comments in TDE/tde-cmake#99. Removing this file therefore seems to be incorrect or, respectively, when merging tqtinterface to TQt3, this file will need to be generated in TQt3. Perhaps with the name `tqui.pc`, which corresponds to the name of the library.
Poster
Owner

This PR is wrong, I did the wrong checking in first place.

This PR is wrong, I did the wrong checking in first place.
MicheleC closed this pull request 3 months ago
MicheleC added the PR/not-ok label 3 months ago
MicheleC removed this from the R14.2.0 release milestone 3 months ago
Poster
Owner

I will make a separate PR for the other files not needed.

I will make a separate PR for the other files not needed.
MicheleC deleted branch drop/unused-code 3 months ago

Reviewers

SlavekB requested changes 3 months ago
TDE/Core was requested for review 3 months ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
TDE/Core
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tqtinterface#39
Loading…
There is no content yet.