Merge pull request #337 from mkroening/cargo-non_kebab_case_bins

fix(tools): Fix `cargo::non_kebab_case_bins`

Author: martin-hughes

.cargo/config.toml

@@ -1,2 +1,2 @@
 [alias]
-run_tests = "run --manifest-path tools/aml_tester/Cargo.toml --"
+run_tests = "run --manifest-path tools/aml-tester/Cargo.toml --"

.github/workflows/build.yml

@@ -93,5 +93,5 @@ jobs:
           rustup default nightly
 
       - name: Build no alloc test exe
-        working-directory: tools/no_alloc_check
+        working-directory: tools/no-alloc-check
         run: cargo build

.gitignore

@@ -5,4 +5,4 @@ tests/*.aml
 tests/*.lst
 tests/*.txt
 dumps/
-tools/no_alloc_check/target
+tools/no-alloc-check/target

Cargo.toml

@@ -1,7 +1,7 @@
 [workspace]
-members = ["tools/aml_tester", "tools/acpi_dumper", "tools/aml_test_tools", "tools/uacpi_test_adapter"]
-# See the note in the no_alloc_check README.md to see why this is excluded.
-exclude = ["tools/no_alloc_check"]
+members = ["tools/aml-tester", "tools/acpi-dumper", "tools/aml-test-tools", "tools/uacpi-test-adapter"]
+# See the note in the no-alloc-check README.md to see why this is excluded.
+exclude = ["tools/no-alloc-check"]
 resolver = "2"
 
 [package]
@@ -24,7 +24,7 @@ byteorder = { version = "1.5.0", optional = true, default-features = false }
 smallvec = { version = "1.15.2", optional = true, default-features = false }
 
 [dev-dependencies]
-aml_test_tools = { path = "tools/aml_test_tools" }
+aml-test-tools = { path = "tools/aml-test-tools" }
 lock_api = "0.4.14"
 parking_lot = "0.12.5"
 pretty_env_logger = "0.5.0"

tests/README.md

@@ -4,7 +4,7 @@ There are 4 categories of tests:
 
 1. Unit tests in the main crate source.
 2. Integration tests in the `tests` directory - these all have the extension `.rs`.
-3. Raw ASL files that can be checked using `aml_tester`
+3. Raw ASL files that can be checked using `aml-tester`
 4. Support for running tests from the `uacpi` project.
 
 ## Unit tests
@@ -15,23 +15,23 @@ relatively few unit tests at present - feel free to send PRs with more!
 ## Integration tests
 
 These are the `.rs` files in the `tests` directory. They largely consist of snippets of ASL code that are parsed and
-then executed using the `aml_test_tools` sub-crate. Most of the tests specify the expected output from the provided
+then executed using the `aml-test-tools` sub-crate. Most of the tests specify the expected output from the provided
 `Handler`, which provides reasonable confidence the parser is working correctly.
 
-The `aml_test_tools` sub-crate provides various utilities for writing this style of test.
+The `aml-test-tools` sub-crate provides various utilities for writing this style of test.
 
 Like the unit tests, they can be run using `cargo test`.
 
 ## Raw ASL files
 
 Various ASL files are provided in the `tests` directory. They can be parsed and executed by running 
-`cargo run_tests -p tests` from the root of the repository. This cargo alias runs the `aml_tester` tool.
+`cargo run_tests -p tests` from the root of the repository. This cargo alias runs the `aml-tester` tool.
 
-The `aml_tester` tool provides a command-line way to run ASL files - similar to `acpiexec`.
+The `aml-tester` tool provides a command-line way to run ASL files - similar to `acpiexec`.
 
 ## Running tests from the `uacpi` project
 
-> See the documentation for `uacpi_test_adapter` for more detailed information. Note that a large proportion of the
+> See the documentation for `uacpi-test-adapter` for more detailed information. Note that a large proportion of the
 > tests in the `uacpi` project do not yet pass in this crate.
  
 The `uacpi` project has a fairly extensive test suite. It makes sense for us to be able to check against their test
@@ -41,10 +41,10 @@ suite as well as our own. This can be done by:
 2. Navigating to the root of that repo.
 3. Running something like: 
    ```shell
-   python3 tests/run_tests.py --test-runner ../acpi/target/debug/uacpi_test_adapter
+   python3 tests/run_tests.py --test-runner ../acpi/target/debug/uacpi-test-adapter
    ```
-   Adjusting the paths as necessary. See the `uacpi_test_adapter` documentation for details on how it finds
-   `aml_tester`.
+   Adjusting the paths as necessary. See the `uacpi-test-adapter` documentation for details on how it finds
+   `aml-tester`.
 
 Note that, at present, several of the uACPI tests run indefinitely. You may want to skip these! The easiest way to do so
 is probably just to delete them... The relevant tests are:

tests/multi_table.asl

@@ -1,5 +1,5 @@
-/* This file is a test for `aml_tester` rather than of the parser itself.
- * Can `aml_tester` cope with multiple tables?
+/* This file is a test for `aml-tester` rather than of the parser itself.
+ * Can `aml-tester` cope with multiple tables?
  */
 DefinitionBlock("", "DSDT", 1, "RSACPI", "BUFFLD", 1) {
     OperationRegion(MEM, SystemMemory, 0x40000, 0x1000)

tools/acpi_dumper/Cargo.toml tools/acpi-dumper/Cargo.tomltools/acpi_dumper/Cargo.toml renamed to tools/acpi-dumper/Cargo.toml

@@ -1,5 +1,5 @@
 [package]
-name = "acpi_dumper"
+name = "acpi-dumper"
 version = "0.0.0"
 publish = false
 authors = ["Isaac Woods", "Matt Taylor"]

tools/acpi_dumper/src/main.rs tools/acpi-dumper/src/main.rstools/acpi_dumper/src/main.rs renamed to tools/acpi-dumper/src/main.rs

@@ -1,5 +1,5 @@
 [package]
-name = "aml_test_tools"
+name = "aml-test-tools"
 version = "0.1.0"
 authors = ["Isaac Woods", "Martin Hughes"]
 edition = "2024"