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.

Sign up for GitHub

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

Jump to bottom

Import Cargo, first steps towards porting to Rust 🦀 #343

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

Import Cargo, first steps towards porting to Rust 🦀 #343

wants to merge 9 commits into from

Commits on Apr 20, 2021

  1. Initial import for Cargo.

    @katef
    katef committed Apr 20, 2021
    3 Configuration menu
    Copy the full SHA
    8d24c4f View commit details
    Browse the repository at this point in the history

  2. Special cases for tests with their own build stuff.

    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    a643036 View commit details
    Browse the repository at this point in the history

  3. An attempt to bring in Cargo-generated dependencies.

    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    00c33df View commit details
    Browse the repository at this point in the history

  4. Run cargo test.

    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    9772089 View commit details
    Browse the repository at this point in the history

  5. Switch to staticlib rather than symlinking for an archive.

    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    6e05435 View commit details
    Browse the repository at this point in the history

  6. Remove support for building shared libraries. 

    This is a shame, but gets in the way of porting to Rust.
    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    34e8b49 View commit details
    Browse the repository at this point in the history

  7. Add -undefined dynamic_lookup for symbols provided by Cargo-built l… 

    …ibraries.
    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    b572d09 View commit details
    Browse the repository at this point in the history

  8. Switch to ld -r linking the cargo-generated .a archive when producing… 

    … the partially linked .o files.

This hopefully avoids an ld bug where `-undefined dynamic_lookup` doesn't work on MacOS, and also removes the need for programs to link both the C and Rust libraries directly.
    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    bb83fd4 View commit details
    Browse the repository at this point in the history

  9. Only the one ${part} depends on libfsm_rs.

    @katef
    katef committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    677279d View commit details
    Browse the repository at this point in the history