workflow: add job summary

Author: dyphire

.github/workflows/mpv.yml

@@ -113,7 +113,7 @@ jobs:
     needs: params
     runs-on: ubuntu-latest
     container:
-      image: archlinux:base-devel
+      image: archlinux/archlinux:base-devel
     strategy:
       matrix:
         bit: [64]
@@ -249,6 +249,78 @@ jobs:
           name: ${{ env.ffmpeg_name }}
           path: mpv-winbuild-cmake/release/${{ env.ffmpeg_name }}.7z
 
+
+      - name: "Job summary"
+        uses: actions/github-script@v6
+        continue-on-error: true
+        if: ${{ always() }}
+        with:
+          script: |
+            const path = require('path');
+            const { readdirSync,existsSync } = require('fs');
+            const myExec = async (command, args = null) => await exec.getExecOutput(command,args,{silent: true}).then(result => result.stdout.trim());
+            
+            const toolchainDir = path.resolve("mpv-winbuild-cmake");
+            const workdir = path.resolve(toolchainDir,"src_packages");
+            const isGitSync = dirname => existsSync(path.join(workdir, dirname, '.git'));
+            
+            const getGithubUrl = (hash,remote) => remote.replace(/\.git$/,"") + `/commit/${hash}`;
+            const getGitlabUrl = (hash,remote) => remote.replace(/\.git$/,"") + `/-/commit/${hash}`;
+            const getBitbucketUrl = (hash,remote) => remote.replace(/\.git$/,"") + `/commits/${hash}`;
+            const getGoogleSourceUrl = (hash,remote) => remote + `/+/${hash}`;            
+            const gethGitVideolanUrl = (hash,remote) => remote.replace(/\/git\//,"/?p=") + `;a=commit;h=${hash}`;
+            const getCgitUrl = (hash,remote) => remote + `/commit/?id=${hash}`;
+            function getCommitUrl(hash,remote) {
+                let url = "";
+                switch (true) {
+                  case /github\.com/.test(remote):
+                    url = getGithubUrl(hash,remote);
+                    break;
+                  case /(gitlab\.com|code\.videolan\.org|gitlab\.gnome\.org)/.test(remote):
+                    url = getGitlabUrl(hash,remote);
+                    break;
+                  case /bitbucket\.org/.test(remote):
+                    url = getBitbucketUrl(hash,remote);
+                    break;
+                  case /googlesource\.com/.test(remote):
+                    url = getGoogleSourceUrl(hash,remote);
+                    break;
+                  case /git\.videolan\.org/.test(remote):
+                    url = gethGitVideolanUrl(hash,remote);
+                    break;
+                  case /git\.libssh\.org/.test(remote):
+                    url = getCgitUrl(hash,remote);
+                    break;
+                  default:
+                    url = remote;
+                    break;
+                }
+                return url;
+            }
+            
+            const dirs = readdirSync(workdir, { withFileTypes: true }).filter(dirent => dirent.isDirectory() && isGitSync(dirent.name) ).map(dirent => path.join(workdir, dirent.name));
+            let packages_table = [[{data: 'Package', header: true}, {data: 'Local commit', header: true}, {data: 'Status', header: true}, {data: 'Remote commit', header: true}]];
+            for (let dir of dirs) {
+              let local_hash = await myExec(`git -C ${dir} rev-parse --short=7 HEAD`);
+              let remote_branch = await myExec(`git -C ${dir} rev-parse --abbrev-ref HEAD@{upstream}`);
+              let remote_hash = await myExec(`git -C ${dir} rev-parse ${remote_branch}`);
+              let status = await myExec(`git -C ${dir} status -sb`).then(s => s.split("\n",1)[0].replace(/^## */,""));
+              let remote = await myExec(`git -C ${dir} config --get remote.origin.url`);
+              let url = getCommitUrl(remote_hash, remote);
+              let package_name = path.basename(dir);
+              packages_table.push([package_name, local_hash, status, `<a href="${url}">${remote_hash.slice(0,7)}</a>`]);
+            }
+            let toolchain_hash = await myExec(`git -C ${toolchainDir} rev-parse HEAD`);
+            let toolchain_remote = await myExec(`git -C ${toolchainDir} config --get remote.origin.url`);
+            let toolchain_url = getCommitUrl(toolchain_hash, toolchain_remote);
+            
+            await core.summary
+            .addHeading('Basic Info')
+            .addRaw(`toolchain: <a href="${toolchain_url}">${toolchain_hash.slice(0,7)}</a>`,true)
+            .addHeading('Packages Version')
+            .addTable(packages_table)
+            .write();
+
   publish_release:
     name: Publish release
     needs: [build_mpv,params]
@@ -293,7 +365,7 @@ jobs:
           artifacts: "artifacts/*/*.7z"
           commit: version
           name: "MPV_OWN ${{ env.long_time }}"
-          body: "**MPV git commit**: https://github.com/dyphire/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n- merged: https://github.com/mpv-player/mpv/pull/8865, https://github.com/mpv-player/mpv/pull/9360, https://github.com/mpv-player/mpv/pull/9621, https://github.com/mpv-player/mpv/pull/9664, https://github.com/mpv-player/mpv/pull/9856\n\n**Test**: Build with libass from [dyphire/libass_dev](https://github.com/dyphire/libass/tree/dev)\n- Ass: it can fix some problems for scaled with PlayResX\n\n**Remember**: This is not perfect\n**Build Time**: ${{ env.long_time }}"
+          body: "**MPV git commit**: https://github.com/dyphire/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n- merged: https://github.com/mpv-player/mpv/pull/8865, https://github.com/mpv-player/mpv/pull/9360, https://github.com/mpv-player/mpv/pull/9621, https://github.com/mpv-player/mpv/pull/9664, https://github.com/mpv-player/mpv/pull/9856\n\n**Test**: Build with libass from [dyphire/libass_dev](https://github.com/dyphire/libass/tree/dev)\n- Ass: it can fix some problems for scaled with PlayResX\n\n**Remember**: This is not perfect\n**Build Details**: https://github.com/${{github.repository}}/actions/runs/${{github.run_id}}\n**Build Time**: ${{ env.long_time }}"
           tag: "mpv_own-${{ env.short_time }}"
           allowUpdates: true
           prerelease: false
@@ -305,7 +377,7 @@ jobs:
           artifacts: "artifacts/*/*.7z"
           commit: version
           name: "MPV_OWN ${{ env.long_time }}"
-          body: "**MPV git commit**: https://github.com/mpv-player/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n**Test**: Build with libass from [dyphire/libass_dev](https://github.com/dyphire/libass/tree/dev)\n- Ass: it can fix some problems for scaled with PlayResX\n\n**Remember**: This is not perfect\n**Build Time**: ${{ env.long_time }}"
+          body: "**MPV git commit**: https://github.com/mpv-player/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n**Test**: Build with libass from [dyphire/libass_dev](https://github.com/dyphire/libass/tree/dev)\n- Ass: it can fix some problems for scaled with PlayResX\n\n**Remember**: This is not perfect\n**Build Details**: https://github.com/${{github.repository}}/actions/runs/${{github.run_id}}\n**Build Time**: ${{ env.long_time }}"
           tag: "mpv_own-${{ env.short_time }}"
           allowUpdates: true
           prerelease: false
@@ -317,7 +389,7 @@ jobs:
           artifacts: "artifacts/*/*.7z"
           commit: version
           name: "MPV ${{ env.long_time }}"
-          body: "**MPV git commit**: https://github.com/mpv-player/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n**Build Time**: ${{ env.long_time }}"
+          body: "**MPV git commit**: https://github.com/mpv-player/mpv/commit/${{ fromJson(needs.params.outputs.params).sha }}\n**Build Details**: https://github.com/${{github.repository}}/actions/runs/${{github.run_id}}\n**Build Time**: ${{ env.long_time }}"
           tag: "mpv-${{ env.short_time }}"
           allowUpdates: true
           prerelease: false

build.sh

@@ -34,13 +34,13 @@ build() {
 
     cmake -DTARGET_ARCH=$arch-w64-mingw32 -DALWAYS_REMOVE_BUILDFILES=ON -DSINGLE_SOURCE_LOCATION=$srcdir -G Ninja -H$gitdir -B$buildroot/build$bit
     ninja -C $buildroot/build$bit download || true
-    if [[ "$(ls -A $buildroot/build$bit/install/bin)" ]]; then
-        ninja -C $buildroot/build$bit update
-    else
+    if [[ ! "$(ls -A $buildroot/build$bit/install/bin)" ]]; then
         ninja -C $buildroot/build$bit gcc
     fi
+    ninja -C $buildroot/build$bit update
     ninja -C $buildroot/build$bit mpv-fullclean
-    ninja -C $buildroot/build$bit mpv || ninja -C $buildroot/build$bit mpv || ninja -C $buildroot/build$bit mpv
+    ninja -C $buildroot/build$bit libjxl && ninja -C $buildroot/build$bit vulkan
+    ninja -C $buildroot/build$bit mpv || ninja -C $buildroot/build$bit mpv
     if [ -d $buildroot/build$bit/mpv-$arch* ] ; then
         echo "Successfully compiled $bit-bit. Continue"
     else