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

SWIG bindings and unit tests for Vars::detect_release() #1804

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

Commits on Oct 28, 2024

  1. swig: Move VarsWeakPtr wpapper to conf module

    The vars.hpp file is included in the conf module and the Vars class is
    wrapped in the conf module. VarsWeakPtr was packaged in the base module.
    Which will bring problems in the following changes.
    
    So the VarsWeakPtr wrapper has been moved to the conf module.
    jrohel committed Oct 28, 2024
    Configuration menu
    Copy the full SHA
    e832b9c View commit details
    Browse the repository at this point in the history
  2. swig: Modify Vars::detect_release to return char *

    The original `Vars::detect_release` returns `std::unique_ptr<std::string>`
    This makes it complicated for wrapping into other languages.
    
    The modified version returns `char *` - a standard C string.
    jrohel committed Oct 28, 2024
    Configuration menu
    Copy the full SHA
    879c690 View commit details
    Browse the repository at this point in the history

Commits on Oct 29, 2024

  1. Configuration menu
    Copy the full SHA
    dd802d6 View commit details
    Browse the repository at this point in the history
  2. Unit tests for Vars::detect_release() SWIG bindings

    Tests in Python, Ruby, Perl.
    
    It only tests the situation where a release cannot be detected.
    To detect an installed release, a system database with the corresponding
    installed packages is required.
    jrohel committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    ec76b7a View commit details
    Browse the repository at this point in the history