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

Changes that suggested on #770 #771

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Changes that suggested on #770 #771

wants to merge 2 commits into from

Commits on Sep 7, 2024

  1. Changes that suggested on gnuradio#770

    Writing to the python helper script moved to `python/volk_modtool/CMakeLists.txt`
    Zaryob authored Sep 7, 2024
    Configuration menu
    Copy the full SHA
    d4afa9e View commit details
    Browse the repository at this point in the history

Commits on Sep 25, 2024

  1. Revert "Changes that suggested on gnuradio#770" and Added way given o…

    …n pull req.
    
    This reverts commit d4afa9e partially.
    
    as @jdemel stated:
    > CMAKE_CURRENT_BINARY_DIR would possibly be the preferred way to use. I'd suspect changing this variable in the original location would already fix your issue. That'd be great. That way we can keep the code in a more generic place that makes it easier to re-use. Also, thanks for your patience. 
    
    I suspected about is there any upper config deletes CMAKE_CURRENT_SOURCE_DIR and CMAKE_CURRENT_BUILD_DIR and it overriding on 'CMakeLists.txt'.
    Zaryob committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    6458a0b View commit details
    Browse the repository at this point in the history