feat/trinity-cmake #212

Merged
SlavekB merged 2 commits from feat/trinity-cmake into master 3 years ago
Collaborator

Since the cmake module has been removed in new packages in favor of installing the trinity-cmake package, I suggest enabling the following changes.
This package is expected to be needed in version 14.0.11 as well as in the live version. At the moment, it is impossible to compile a live version of TDE without these changes.

Since the cmake module has been removed in new packages in favor of installing the `trinity-cmake` package, I suggest enabling the following changes. This package is expected to be needed in version 14.0.11 as well as in the live version. At the moment, it is impossible to compile a live version of TDE without these changes.
SlavekB requested changes 3 years ago
SlavekB left a comment
Owner

It's a good step forward, but there is a need for adjustment.

It's a good step forward, but there is a need for adjustment.
[[ ${PN} != tdelibs ]] && [[ ${TRINITY_MODULE_TYPE} != "dependencies" ]] &&
COMMON_DEPEND+=" ~trinity-base/tdelibs-${PV}"
[[ ${PN} == "14.0.1"[1-9] ]] || [[ "${BUILD_TYPE}" == "live" ]] &&
[[ ${TRINITY_MODULE_TYPE} != "dependencies" ]] &&
Owner

You can notice that Trinity macros for CMake are also used in modules with the type dependencies. Therefore, I believe that this needs to be adjusted.

You can notice that Trinity macros for CMake are also used in modules with the type `dependencies`. Therefore, I believe that this needs to be adjusted.
Poster
Collaborator

Yes, quite right, I corrected it. Replaced with this line:

[[ ${TRINITY_MODULE_NAME} != "tde-common-cmake" ]] &&

so that there are no cyclic dependencies.

Yes, quite right, I corrected it. Replaced with this line: ``` [[ ${TRINITY_MODULE_NAME} != "tde-common-cmake" ]] && ``` so that there are no cyclic dependencies.
SlavekB marked this conversation as resolved
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE pkgmetadata SYSTEM "http://www.gentoo.org/dtd/metadata.dtd">
<pkgmetadata>
<!-- maintainer-needed -->
Owner

BTW, for some time here is a mailing list tde-team-gentoo – email address team-gentoo@trinitydesktop.org. This could be used for metadata. You can consider signing in to this ML.

BTW, for some time here is a mailing list tde-team-gentoo – email address `team-gentoo@trinitydesktop.org`. This could be used for metadata. You can consider signing in to this ML.
Poster
Collaborator

Fixed.

Fixed.
SlavekB marked this conversation as resolved
ormorph force-pushed feat/trinity-cmake from 33801c6f63 to ddf7333488 3 years ago
Poster
Collaborator

Made changes, everything is working.

Made changes, everything is working.
SlavekB approved these changes 3 years ago
SlavekB left a comment
Owner

It looks good.

It looks good.
SlavekB merged commit ddf7333488 into master 3 years ago
SlavekB deleted branch feat/trinity-cmake 3 years ago
SlavekB added this to the R14.0.11 release milestone 3 years ago

Reviewers

SlavekB approved these changes 3 years ago
The pull request has been merged as ddf7333488.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tde-packaging-gentoo#212
Loading…
There is no content yet.