Pacman hook file prefix should be higher than 10

[vestingz]

Pacman hook for regenerating the graph should possibly run after other PostTransaction hooks, which might for example handle .pacnew files. According to man 5 alpm-hooks the running order is defined by the alphabetical order of the hooks filenames, so 10-pacwall.hook is one of the first hooks to be triggered, while it should actually be the last. Renaming it to something like 99-pacwall.hook might be warranted.

[Kharacternyk]

The hook starts a pacwall process and doesn't wait for it to exit. I chose a small number to utilize this asynchronous execution and update the wallpaper ASAP.

I haven't ever thought about hooks that deal with .pacnew files. There is indeed a race condition.

Are there some well-known hooks that interact with .pacnew files? If so, we could choose a number that is just above ones of such hooks.

99-pacwall.hook solves the issue, but it worsens the responsiveness, albeit by a tiny bit.

[vestingz]

Yes, this makes sense, too.

Are there some well-known hooks that interact with .pacnew files? If so, we could choose a number that is just above ones of such hooks.

I do not know about any specific well-known hook, but I do have a custom hook to remove the `.pacnew` file after a `pacman-mirrorlist` update. Generating a new graph through the pacwall hook prematurely wrongly shows that package to be unresolved in the graph.

`99-pacwall.hook` solves the issue, but it worsens the responsiveness, albeit by a tiny bit.

Yeah, true. Maybe in the end it might be helpful to have this edge case documented in the README or so..

[Kharacternyk]

I propose changing the priority to 11, so that hooks like the one you use can be run before pacwall and adding a note to the README. It works as long as the hook you use executes properly if run before all the other hooks, though.