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

More tests and other fixes #35

Merged
merged 4 commits into from
Aug 14, 2024
Merged

More tests and other fixes #35

merged 4 commits into from
Aug 14, 2024

Commits on Aug 6, 2024

  1. tests: add simple test case for jar files

    Test case created with:
    $ javac hello.java
    $ jar -cf hello.jar hello.class
    
    "Fixed" version is our output.
    keszybz committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    4764673 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2024

  1. Rename options.rs to config.rs

    The primary exported name is Config, so it's better to call this
    config.
    keszybz committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    489e7af View commit details
    Browse the repository at this point in the history
  2. multiprocess: drop OwnedFds instead of closing them

    The tests started failing with:
    fatal runtime error: IO Safety violation: owned file descriptor already closed
    (rust-1.80.0-1.fc40.x86_64)
    
    It seems that we were trying to close the fd from two places
    and somehow this didn't cause an error before.
    keszybz committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    c6d9676 View commit details
    Browse the repository at this point in the history
  3. Prefix logs with [pid] when invoked with -j

    With interleaved logs from many workers, it can be hard to to figure
    out which worker is logging what.
    keszybz committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    ac18782 View commit details
    Browse the repository at this point in the history