Fix reporting of the version number.

You need fetch-depth: 0 to get the history. Then you need the right
arguments to `git describe` to avoid it returning some older tag.

Author: jonathanpallant

.github/workflows/build_and_release.yml

@@ -1,15 +1,19 @@
 on: [push, pull_request]
 name: Build (and Release)
 
+env:
+  CARGO_TERM_COLOR: always
+
 jobs:
   build:
     name: Build (and Release)
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v1
+        uses: actions/checkout@v3
         with:
           submodules: true
+          fetch-depth: 0
 
       - name: Add targets
         run: |
@@ -28,24 +32,6 @@ jobs:
       - name: Build neotron-bmc-nucleo
         run: cd neotron-bmc-nucleo && DEFMT_LOG=info cargo build --release --verbose --target=thumbv7em-none-eabihf
 
-      - name: Get Branch Name
-        if: github.event_name == 'push' && startswith(github.ref, 'refs/tags/')
-        id: branch_name
-        run: |
-          echo ::set-output name=SOURCE_TAG::${GITHUB_REF#refs/tags/}
-
-      - name: Create Release
-        if: github.event_name == 'push' && startswith(github.ref, 'refs/tags/')
-        id: create_release
-        uses: actions/create-release@v1
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          tag_name: ${{ github.ref }}
-          release_name: Release ${{ steps.branch_name.outputs.SOURCE_TAG }}
-          draft: false
-          prerelease: false
-
       - name: Upload files to Release
         if: github.event_name == 'push' && startswith(github.ref, 'refs/tags/')
         uses: softprops/action-gh-release@v1

neotron-bmc-pico/build.rs

@@ -19,9 +19,15 @@ fn main() {
 	// Generate a file containing the firmware version
 	let version_output = std::process::Command::new("git")
 		.current_dir(env::var_os("CARGO_MANIFEST_DIR").unwrap())
-		.args(["describe", "--long", "--dirty"])
+		.args(["describe", "--tags", "--dirty"])
 		.output()
 		.expect("running git-describe");
+
+	println!(
+		"Version is {:?}",
+		std::str::from_utf8(&version_output.stdout)
+	);
+	println!("Error is {:?}", std::str::from_utf8(&version_output.stderr));
 	assert!(version_output.status.success());
 
 	// Remove the trailing newline
@@ -32,6 +38,7 @@ fn main() {
 		panic!("Version too long!");
 	}
 
+	// Pad it
 	while output.len() < 32 {
 		output.push(0);
 	}