`--with-homedir` gives "yes" as config directory

[gnusupport]

Configure options:

./configure --prefix=/package/admin/mc --enable-charset --enable-aspell --enable-doxygen-man --with-search-engine=pcre --with-x --enable-doxygen-pdf --enable-vfs-undelfs --with-gpm-mouse --with-homedir

end result

[User data]
    Config directory: /root/yes/
    Data directory:   /root/yes/
        skins:          /root/yes/skins/
        extfs.d:        /root/yes/extfs.d/
        fish:           /root/yes/fish/
        mcedit macros:  /root/yes/mc.macros
        mcedit external macros: /root/yes/mcedit/macros.d/macro.*
    Cache directory:  /root/yes/

I am not sure in that, I am going to try without the option --with-homedir however, all configuration files are now in ~/yes/ directory

Using git version: GNU Midnight Commander 4.8.19-65-g5e5f57c

[zaytsev]

Folks, could you please review the attached patch? It sets the default --with-homedir value to .mc instead of yes, so that one can just use --with-homedir instead of --with-homedir=.mc. May I commit this?

[gnusupport]

I am in difficult place to re-compile it all. If you think it is good, is good.

[andrew_b]

The root of problem is using of standard autoconf AC_ARG_WITH macro for directory name, because --with-* allow have no argument. If argument is absent, then argument is yes. It seems, autoconf doesn't provide a macro like AC_ARG_WITH with mandatory argument.

[zaytsev]

Andrew, you are right, but in this particular case, we can just set the default (yes) to .mc to solve the problem like I did, can't we? This should be appropriate in 99% of the cases. Alternatively, we can just AC_MSG_ERROR if no argument is provided (the value is yes), there is no need for an extended macro, really, but I don't feel like this would be a superior approach from the user experience PoV. WDYT?

I am in difficult place to re-compile it all.

This takes no longer than 5 minutes (tm), so what seems to be the problem?

[andrew_b]

Replying to zaytsev:

Alternatively, we can just AC_MSG_ERROR if no argument is provided (the value is yes),

Actually, the --with-homedir and --with-homedir=yes are same cases.

I think you should add some comments in your patch about this situation.

[zaytsev]

You are right, a comment would be a good idea. How about now?

In as far as the corner case --with-homedir=yes is concerned, I think that if anyone would really seriously want to do that she/he should be happy enough with a workaround like --with-homedir=./yes ;-)

[zaytsev]

Branch: 3836_with_homedir_yes
Initial changeset: [9932087cec6b7e539556203f1ea3335e6546236a]

[zaytsev]

Merged to master: [eab2556ed8e9649401ff7a629ebcf0a25adea078]