helpers

.github/workflows/clippy.yml

@@ -31,6 +31,8 @@ jobs:
         uses: ./.github/actions/fix-environment
       - name: Clippy cppwinrt
         run:  cargo clippy -p cppwinrt
+      - name: Clippy helpers
+        run:  cargo clippy -p helpers
       - name: Clippy riddle
         run:  cargo clippy -p riddle
       - name: Clippy sample_bits
@@ -269,8 +271,6 @@ jobs:
         run:  cargo clippy -p tool_bindings
       - name: Clippy tool_gnu
         run:  cargo clippy -p tool_gnu
-      - name: Clippy tool_lib
-        run:  cargo clippy -p tool_lib
       - name: Clippy tool_license
         run:  cargo clippy -p tool_license
       - name: Clippy tool_msvc

.github/workflows/no-default-features.yml

@@ -29,6 +29,8 @@ jobs:
         uses: ./.github/actions/fix-environment
       - name: Check cppwinrt
         run:  cargo check -p cppwinrt --no-default-features
+      - name: Check helpers
+        run:  cargo check -p helpers --no-default-features
       - name: Check windows
         run:  cargo check -p windows --no-default-features
       - name: Check windows-bindgen

.github/workflows/test.yml

@@ -55,6 +55,8 @@ jobs:
         run:  cargo clean
       - name: Test cppwinrt
         run:  cargo test -p cppwinrt --target ${{ matrix.target }} ${{ matrix.etc }}
+      - name: Test helpers
+        run:  cargo test -p helpers --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test riddle
         run:  cargo test -p riddle --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test sample_bits
@@ -151,10 +153,10 @@ jobs:
         run:  cargo test -p test_agile_reference --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_alternate_success_code
         run:  cargo test -p test_alternate_success_code --target ${{ matrix.target }} ${{ matrix.etc }}
-      - name: Test test_arch
-        run:  cargo test -p test_arch --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Clean
         run:  cargo clean
+      - name: Test test_arch
+        run:  cargo test -p test_arch --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_arch_feature
         run:  cargo test -p test_arch_feature --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_array
@@ -253,10 +255,10 @@ jobs:
         run:  cargo test -p test_reserved --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_resources
         run:  cargo test -p test_resources --target ${{ matrix.target }} ${{ matrix.etc }}
-      - name: Test test_result
-        run:  cargo test -p test_result --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Clean
         run:  cargo clean
+      - name: Test test_result
+        run:  cargo test -p test_result --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_return_handle
         run:  cargo test -p test_return_handle --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test test_return_struct
@@ -297,8 +299,6 @@ jobs:
         run:  cargo test -p tool_bindings --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test tool_gnu
         run:  cargo test -p tool_gnu --target ${{ matrix.target }} ${{ matrix.etc }}
-      - name: Test tool_lib
-        run:  cargo test -p tool_lib --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test tool_license
         run:  cargo test -p tool_license --target ${{ matrix.target }} ${{ matrix.etc }}
       - name: Test tool_msvc

Cargo.toml

@@ -30,3 +30,4 @@ windows-strings = { path = "crates/libs/strings" }
 windows-sys = { path = "crates/libs/sys" }
 windows-targets = { path = "crates/libs/targets" }
 windows-version = { path = "crates/libs/version" }
+helpers = { path = "crates/libs/helpers" }

crates/tools/lib/Cargo.toml → crates/libs/helpers/Cargo.toml

@@ -1,5 +1,5 @@
 [package]
-name = "tool_lib"
+name = "helpers"
 version = "0.0.0"
 edition = "2021"
 publish = false

crates/tools/lib/src/lib.rs → crates/libs/helpers/src/lib.rs

@@ -10,18 +10,12 @@ pub enum CallingConvention {
 
 pub fn default_metadata() -> Vec<metadata::File> {
     vec![
-        metadata::File::new(
-            std::include_bytes!("../../../libs/bindgen/default/Windows.winmd").to_vec(),
-        )
-        .expect("invalid winmd"),
-        metadata::File::new(
-            std::include_bytes!("../../../libs/bindgen/default/Windows.Win32.winmd").to_vec(),
-        )
-        .expect("invalid winmd"),
-        metadata::File::new(
-            std::include_bytes!("../../../libs/bindgen/default/Windows.Wdk.winmd").to_vec(),
-        )
-        .expect("invalid winmd"),
+        metadata::File::new(std::include_bytes!("../../bindgen/default/Windows.winmd").to_vec())
+            .expect("invalid winmd"),
+        metadata::File::new(std::include_bytes!("../../bindgen/default/Windows.Win32.winmd").to_vec())
+            .expect("invalid winmd"),
+        metadata::File::new(std::include_bytes!("../../bindgen/default/Windows.Wdk.winmd").to_vec())
+            .expect("invalid winmd"),
     ]
 }
 

crates/tests/metadata/Cargo.toml

@@ -25,5 +25,5 @@ features = [
     "Win32_System_SystemServices",
 ]
 
-[dependencies.tool_lib]
-path = "../../tools/lib"
+[dependencies.helpers]
+workspace = true

crates/tests/metadata/tests/attribute_enum.rs

@@ -2,7 +2,7 @@ use windows_metadata::*;
 
 #[test]
 fn attribute_enum() {
-    let files = tool_lib::default_metadata();
+    let files = helpers::default_metadata();
     let reader = Reader::new(files);
 
     let (method, _) = reader

crates/tests/metadata/tests/fn_call_size.rs

@@ -5,7 +5,7 @@ fn size() {
     // Note: you can double check these export names from a Visual Studio x86 command prompt as follows:
     // dumpbin /exports kernel32.lib | findstr /i RtmConvertIpv6AddressAndLengthToNetAddress
 
-    let files = tool_lib::default_metadata();
+    let files = helpers::default_metadata();
     let reader = Reader::new(files);
 
     assert_eq!(

crates/tests/metadata/tests/unused.rs

@@ -2,7 +2,7 @@ use windows_metadata::*;
 
 #[test]
 fn test() {
-    let files = tool_lib::default_metadata();
+    let files = helpers::default_metadata();
     let reader = Reader::filter(
         files,
         &["Windows", "BadNamespace", "Windows.AI"],

crates/tests/riddle/Cargo.toml

@@ -14,8 +14,8 @@ workspace = true
 [dependencies.windows-metadata]
 workspace = true
 
-[dependencies.tool_lib]
-path = "../../tools/lib"
+[dependencies.helpers]
+workspace = true
 
 [dependencies.windows-bindgen]
 workspace = true

crates/tests/riddle/src/lib.rs

@@ -64,7 +64,7 @@ pub fn run_riddle(name: &str, dialect: &str, etc: &[&str]) -> Vec<windows_metada
     run_one_bindgen(&command);
 
     // Return winmd file for validation
-    let mut files = tool_lib::default_metadata();
+    let mut files = helpers::default_metadata();
     let winmd_bytes =
         std::fs::read(&winmd).unwrap_or_else(|e| panic!("Failed to read winmd: {winmd} : {e:?}"));
     files.push(

crates/tests/targets/Cargo.toml

@@ -19,5 +19,5 @@ features = [ "Win32_Security_Authentication_Identity" ]
 workspace = true
 features = [ "Win32_Security_Authentication_Identity" ]
 
-[dependencies.tool_lib]
-path = "../../tools/lib"
+[dependencies.helpers]
+workspace = true

crates/tests/targets/tests/files.rs

@@ -3,7 +3,7 @@
 
 #[test]
 fn test() {
-    let targets = tool_lib::crates("../../targets");
+    let targets = helpers::crates("../../targets");
     assert_eq!(8, targets.len());
     assert!(targets.iter().all(|(_, version)| version == "0.52.6"));
 

crates/tests/winrt/Cargo.toml

@@ -9,7 +9,7 @@ doc = false
 doctest = false
 
 [dependencies.windows]
-path = "../../libs/windows"
+workspace = true
 features = [
     "implement",
     "AI_MachineLearning",

crates/tools/gnu/Cargo.toml

@@ -4,5 +4,5 @@ version = "0.52.0"
 edition = "2021"
 publish = false
 
-[dependencies]
-lib = { package = "tool_lib", path = "../lib" }
+[dependencies.helpers]
+workspace = true

crates/tools/msvc/Cargo.toml

@@ -4,5 +4,5 @@ version = "0.52.0"
 edition = "2021"
 publish = false
 
-[dependencies]
-lib = { package = "tool_lib", path = "../lib" }
+[dependencies.helpers]
+workspace = true

crates/tools/yml/Cargo.toml

@@ -4,5 +4,5 @@ version = "0.0.0"
 edition = "2021"
 publish = false
 
-[dependencies]
-lib = { package = "tool_lib", path = "../lib" }
+[dependencies.helpers]
+workspace = true

crates/tools/yml/src/main.rs

@@ -65,7 +65,7 @@ jobs:
     // This unrolling is required since "cargo test --all" consumes too much memory for the GitHub hosted runners
     // and the occasional "cargo clean" is required to avoid running out of disk space in the same runners.
 
-    for (count, (name, _)) in lib::crates("crates").iter().enumerate() {
+    for (count, (name, _)) in helpers::crates("crates").iter().enumerate() {
         if count % 50 == 0 {
             write!(
                 &mut yml,
@@ -136,7 +136,7 @@ jobs:
 
     // This unrolling is required since "cargo clippy --all" consumes too much memory for the GitHub hosted runners.
 
-    for (name, _) in lib::crates("crates") {
+    for (name, _) in helpers::crates("crates") {
         write!(
             &mut yml,
             r"
@@ -181,7 +181,7 @@ jobs:
         uses: ./.github/actions/fix-environment"
         .to_string();
 
-    for (name, _) in lib::crates("crates/libs") {
+    for (name, _) in helpers::crates("crates/libs") {
         write!(
             &mut yml,
             r"