Arch Linux

sorry, the task type should be "feature request", not "bug report"

+1 to adding this feature.

Some comments on your patch:

* I'd rather prefer patching makechrootpkg to accept more than one package first and adding this feature in a separate patch. Having to invoke makechrootpkg a couple of times is a bad idea and bad style.

* Please use `git format-patch` to create patches with meaningful commit messages and author information. Also, arch-dev-projects might be a good (better?) place for such a patch.

"+1 to adding this feature."
Yay :)

"Some comments on your patch:

* I'd rather prefer patching makechrootpkg to accept more than one package first and adding this feature in a separate patch. Having to invoke makechrootpkg a couple of times is a bad idea and bad style."
Agreed.
Patch to makechrootpkg to add multiple -I flag support attached.
Also attached the patch to archbuild to add multiple -I flag support which takes into account the patch to makechrootpkg.

"* Please use `git format-patch` to create patches with meaningful commit messages and author information. Also, arch-dev-projects might be a good (better?) place for such a patch."
Attached the proper patches using git format-patch. I like posting these feature requests since they aren't as easily forgotten about imho. I've posted a couple of other patches to devtools the same way in the past and I haven't gotten called out on it.

Suggestions:

* Use double brackets ("[["/"]]") instead of the "[" builtin. We use these almost everywhere in devtools - makechrootpkg is one of the scripts that doesn't. heftig has a patch for this in his working branch iirc. You might need to rebase your patch on that, also.

* Use "${foo##*/}" instead of invoking basename(1).

* 'IFS=" " mkarchroot -r "pacman -U ${pkgnames[*]} --noconfirm" "$copydir"' might break if any of the package file names contain a whitespace. We will probably need to patch mkarchroot in order to being able to fix that.

"* Use double brackets ("[["/"]]") instead of the "[" builtin. We use these almost everywhere in devtools - makechrootpkg is one of the scripts that doesn't. heftig has a patch for this in his working branch iirc. You might need to rebase your patch on that, also."
Couldn't have possibly known this especially since his branch is on github.
If I rebase it, it will mean that I will have to wait for his changes to get merged.
If that's what you want, then I will just wait until his patches get pulled into devtools since they might not be final.

"* Use "${foo##*/}" instead of invoking basename(1)."
Again, I was just sticking to what was there originally and I do find that using basename is a lot easier to read and figure out what it's doing imo.
I will switch it like you suggested.

* 'IFS=" " mkarchroot -r "pacman -U ${pkgnames[*]} --noconfirm" "$copydir"' might break if any of the package file names contain a whitespace. We will probably need to patch mkarchroot in order to being able to fix that."
Why not just include quotes around each of the pkgnames?
pkgnames=("'pkgname1'" "'pkgname2'")

It's been almost a year and a half since I've submitted this patch.

Can we please make a decision?

I still find it beneficial to have this patch.

But I would like to have a decision made within a reasonable amount of time.