commit-changed-pkgs: Take message as argument

[hololeap]

Instead of keeping the message in a file, take the message as an argument.

Automatically prepends *: to the message.

[thesamesam]

How does this work if I want to write something with multiple lines?

[hololeap]

How does this work if I want to write something with multiple lines?

I didn't think of that possibility. I'll go ahead and close this.

[thesamesam]

I don't think you need to. If we pass in additional arguments to git (pkgdev passes any unknown args to git), it might work, or we can file a bug for pkgdev asking it to allow template-then-append if it doesn't already?

[hololeap]

Here's another QoL issue we might want to do something about:

• dev-haskell/df1 gentoo-haskell/gentoo-haskell#1411 (comment)

  sam_ has a script that will create separate commits for many added/modified packages:
  https://github.com/thesamesam/sam-gentoo-scripts/blob/main/git/commit-changed-pkgs

  However, this usually takes a template message in a file (instead of auto-generating a commit message) so you'll probably want to modify it:

  @@ -52,7 +52,7 @@ for dir in ${dirs[@]} ; do
          cd "${dir}"
   
          git add .
  -       pkgdev commit -M ${template}
  +       pkgdev commit
   
          cd ../..
   done

[hololeap]

Another pain point:

sam-gentoo-scripts/git/commit-changed-pkgs

Line 23 in 715ff7c

list=$(git diff --name-only | grep -v "/layout/" | grep -v "/metadata/")

git diff --name-only does not list untracked files, so the script doesn't work for committing brand new packages. Perhaps that wasn't the script's original purpose, but it seems like a similar enough use case for the script to support it.

This command lists untracked directories, fwiw:

git ls-files --others --directory --exclude-standard