Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New package: kwave-22.08.2 #38388

Merged
merged 1 commit into from Oct 30, 2022
Merged

New package: kwave-22.08.2 #38388

merged 1 commit into from Oct 30, 2022

Conversation

hervyqa
Copy link
Contributor

@hervyqa hervyqa commented Jul 31, 2022

Fix: #37680

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64)

Screenshot_20220803_095532

Error: CMakeOutput.log

@hervyqa hervyqa marked this pull request as draft July 31, 2022 10:14
@hervyqa hervyqa force-pushed the kwave branch 4 times, most recently from 9b22684 to 7754e97 Compare July 31, 2022 13:18
@classabbyamp classabbyamp added the new-package This PR adds a new package label Jul 31, 2022
@hervyqa
Copy link
Contributor Author

hervyqa commented Aug 1, 2022

the error is always the same, even though kdoctools (hostmakedepends) and kdoctools-devel (makedepends) already exist. any review? cc @classabbyamp @Johnnynator

 -- Enabled handbook for en
CMake Error at cmake/KwaveHandbook.cmake:51 (GET_TARGET_PROPERTY):
  get_target_property() called with non-existent target "meinproc5".
Call Stack (most recent call first):
  doc/en/CMakeLists.txt:25 (INCLUDE)


CMake Error at doc/CMakeLists.txt:170 (GET_TARGET_PROPERTY):
  get_target_property() called with non-existent target "meinproc5".

@Chocimier
Copy link
Member

Included cmake files looks not in sync with kdoctools modules. Can be patched with:

post_patch() {
    vsed -i doc/CMakeLists.txt -i cmake/KwaveHandbook.cmake \
     -e '/${KDOCTOOLS_MEINPROC_EXECUTABLE}/d' \
     -e 's/${MEINPROC_EXECUTABLE}/meinproc5/'
}

@hervyqa hervyqa force-pushed the kwave branch 2 times, most recently from af543c3 to 7768543 Compare August 3, 2022 00:36
@hervyqa
Copy link
Contributor Author

hervyqa commented Aug 3, 2022

Thanks. succeed..
@Chocimier

@hervyqa hervyqa marked this pull request as ready for review August 3, 2022 00:57
@hervyqa hervyqa changed the title New package: kwave-22.04.3 New package: kwave-22.08.0 Aug 19, 2022
@hervyqa hervyqa changed the title New package: kwave-22.08.0 New package: kwave-22.08.1 Sep 25, 2022
@hervyqa hervyqa changed the title New package: kwave-22.08.1 New package: kwave-22.08.2 Oct 25, 2022
@Johnnynator Johnnynator merged commit b277714 into void-linux:master Oct 30, 2022
@hervyqa hervyqa deleted the kwave branch November 1, 2022 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package request: Kwave
4 participants