Nice, thanks! Pushed:

To git.sr.ht:~emersion/wayland.emersion.fr
   1fbf1a3c5912..b6569e7e0df1  master -> master

Pushed, thanks!

To git.sr.ht:~emersion/drmdb
   23cd002cfad8..971340a0d0b7  master -> master

Pushed:

To git.sr.ht:~emersion/mrsh
   c390c40573dd..2f5dcdd7139f  master -> master

Thanks!

Pushed:

To git.sr.ht:~emersion/mrsh
   ac983c4b7ce0..c390c40573dd  master -> master

Thanks!

Hi, thanks for your patch!

The idea looks good, but I'm not really a fan of adding a new
parse_config_arguments return code to re-load a different configuration
file.

I think it'd make more sense to process arguments in two passes: first
process the normal flags (such as --help, --config), then load the
config file, then process the style flags.

What do you think?

Do we need to URL-encode?

Disregard, I forgot to check whether someone already submitted a patch
for this.

---
 meson.build      |  1 +
 src/dirname.c    | 92 ++++++++++++++++++++++++++++++++++++++++++++++++
 test/dirname     | 58 ++++++++++++++++++++++++++++++
 test/meson.build |  1 +
 4 files changed, 152 insertions(+)
 create mode 100644 src/dirname.c
 create mode 100644 test/dirname

diff --git a/meson.build b/meson.build
index 80100fe189cf..51951863c89d 100644
--- a/meson.build
+++ b/meson.build
@@ -19,6 +19,7 @@ oneshots = [
[message trimmed]

Pushed with a minor edit to remove unneeded headers:

To git.sr.ht:~emersion/mrsh
   c86e6d08f435..7c20ded166ac  master -> master

Thanks!

On Friday, September 6, 2019 12:08 AM, Drew DeVault <sir@cmpwn.com> wrote:

> On Thu Sep 5, 2019 at 2:13 PM Simon Ser wrote:
>
> > However the user still can set PWD manually. I wouldn't want users to source
> > a bad script, and then have their shell die because undefined behaviour.
>
> They cannot, it's read-only.

Oh, forgot about that!

> > > > If this is unspecified behaviour, better to error out properly.
> > >
> > > Also a programming error in mrsh.