unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
@ 2023-08-10  2:06 Mou Tong
  2023-08-10  2:37 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 8+ messages in thread
From: Mou Tong @ 2023-08-10  2:06 UTC (permalink / raw)
  To: 65190

After the android branch merged, the file `configure.ac`'s section about
`mailutils` changed.

When I build Emacs on macOS with `./configure`, cli will remind me:

```
configure: WARNING: This configuration installs a 'movemail' program
that does not retrieve POP3 email.  By default, Emacs 25 and earlier
installed a 'movemail' program that retrieved POP3 email via only
insecure channels, a practice that is no longer recommended but that
you can continue to support by using './configure --with-pop'.
configure: You might want to use './configure --with-mailutils'.
```

I have installed mailtuils by homebrew (a package manager on macOS), and
it works fine before the android branch got merged.

I guess the problem occurs in this line

https://git.savannah.gnu.org/cgit/emacs.git/tree/configure.ac?id=e1d08e9e11d2bd5e93c8ae4ea4a8e27129b316cb#n349

But I don't know how to fix this :(





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-10  2:06 bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS Mou Tong
@ 2023-08-10  2:37 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-08-10  3:10   ` Mou Tong
  0 siblings, 1 reply; 8+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-08-10  2:37 UTC (permalink / raw)
  To: Mou Tong; +Cc: 65190

Mou Tong <mou.tong@outlook.com> writes:

> After the android branch merged, the file `configure.ac`'s section about
> `mailutils` changed.
>
> When I build Emacs on macOS with `./configure`, cli will remind me:
>
> ```
> configure: WARNING: This configuration installs a 'movemail' program
> that does not retrieve POP3 email.  By default, Emacs 25 and earlier
> installed a 'movemail' program that retrieved POP3 email via only
> insecure channels, a practice that is no longer recommended but that
> you can continue to support by using './configure --with-pop'.
> configure: You might want to use './configure --with-mailutils'.
> ```
>
> I have installed mailtuils by homebrew (a package manager on macOS), and
> it works fine before the android branch got merged.
>
> I guess the problem occurs in this line
>
> https://git.savannah.gnu.org/cgit/emacs.git/tree/configure.ac?id=e1d08e9e11d2bd5e93c8ae4ea4a8e27129b316cb#n349
>
> But I don't know how to fix this :(

Does the problem resolve itself if you change:

     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
             && test "$with_android" = "no"],
       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],


to read

     [AS_IF([test "x$XCONFIGURE" != "xandroid"],
       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],

?





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-10  2:37 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-08-10  3:10   ` Mou Tong
  2023-08-10  4:47     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 8+ messages in thread
From: Mou Tong @ 2023-08-10  3:10 UTC (permalink / raw)
  To: Po Lu; +Cc: 65190@debbugs.gnu.org

Thx for the quick response, here is the modification:

```diff
--- a/configure.ac
+++ b/configure.ac
@@ -347,8 +347,10 @@ AC_DEFUN
   [],
   [with_mailutils=$with_features
    AS_IF([test "$with_mailutils" = yes],
-     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
-             && test "$with_android" = "no"],
+     dnl [AS_IF([test "x$XCONFIGURE" != "xandroid" \
+     dnl         && test "$with_android" = "no"],
+     dnl   [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
+     [AS_IF([test "x$XCONFIGURE" != "xandroid"],
        [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
        [dnl Don't check for movemail if cross-compiling.
         dnl instead, default to false.
```

But the problem still exists.






^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-10  3:10   ` Mou Tong
@ 2023-08-10  4:47     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-08-10  6:04       ` Mou Tong
  0 siblings, 1 reply; 8+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-08-10  4:47 UTC (permalink / raw)
  To: Mou Tong; +Cc: 65190@debbugs.gnu.org

Mou Tong <mou.tong@outlook.com> writes:

> Thx for the quick response, here is the modification:
>
> ```diff
> --- a/configure.ac
> +++ b/configure.ac
> @@ -347,8 +347,10 @@ AC_DEFUN
>    [],
>    [with_mailutils=$with_features
>     AS_IF([test "$with_mailutils" = yes],
> -     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
> -             && test "$with_android" = "no"],
> +     dnl [AS_IF([test "x$XCONFIGURE" != "xandroid" \
> +     dnl         && test "$with_android" = "no"],
> +     dnl   [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
> +     [AS_IF([test "x$XCONFIGURE" != "xandroid"],
>         [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
>         [dnl Don't check for movemail if cross-compiling.
>          dnl instead, default to false.
> ```
>
> But the problem still exists.

What if you try moving:

AC_ARG_WITH([mailutils],
  [AS_HELP_STRING([--with-mailutils],
     [rely on GNU Mailutils, so that the --without-pop through --with-mailhost
      options are irrelevant; this is the default if GNU Mailutils is
      installed])],
  [],
  [with_mailutils=$with_features
   AS_IF([test "$with_mailutils" = yes],
     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
             && test "$with_android" = "no"],
       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
       [dnl Don't check for movemail if cross-compiling.
        dnl instead, default to false.
        with_mailutils=no])])])
AS_IF([test "$with_mailutils" = no],
  [with_mailutils=])

AS_IF([test x"$with_mailutils" = xyes],
  [AC_DEFINE([HAVE_MAILUTILS], [1],
     [Define to 1 if Emacs was configured with mailutils])])

AC_SUBST([with_mailutils])

AC_ARG_WITH([pop],
  [AS_HELP_STRING([--with-pop],
     [Support POP mail retrieval if Emacs movemail is used (not recommended,
      as Emacs movemail POP is insecure).  This is the default only on
      native MS-Windows and Android.])],
  [],
  dnl Enable movemail POP support on Android as GNU Mailutils is
  dnl normally unavailable on that platform.
  [AS_CASE([$host],
     [*-mingw*|*android*], [with_pop=yes],
     [with_pop=no-by-default])])
if test "$with_pop" = yes; then
   AC_DEFINE([MAIL_USE_POP])
fi
AH_TEMPLATE([MAIL_USE_POP], [Define to support POP mail retrieval.])dnl

to below:

OPTION_DEFAULT_OFF([android],[cross-compile Android application package])
OPTION_DEFAULT_ON([android-debug],[don't build Emacs as a debug package on Android])

?  Does that resolve the problem?

Thanks.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-10  4:47     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-08-10  6:04       ` Mou Tong
  2023-08-11  1:00         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 8+ messages in thread
From: Mou Tong @ 2023-08-10  6:04 UTC (permalink / raw)
  To: Po Lu; +Cc: 65190@debbugs.gnu.org

Ah, sorry.  I forgot to run `./autogen.sh` again to generate a new
`configure`.

After change `configure.ac` like you said before, and exec
`./autogen.sh` to generate a new `configure`. Running `./configure`
works as I expected.

Po Lu <luangruo@yahoo.com> writes:

> Does the problem resolve itself if you change:
>
>     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
>             && test "$with_android" = "no"],
>       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
>
>
> to read
>
>     [AS_IF([test "x$XCONFIGURE" != "xandroid"],
>       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
>
> ?

This had solved the problem ;)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-10  6:04       ` Mou Tong
@ 2023-08-11  1:00         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-08-11  1:37           ` Mou Tong
  0 siblings, 1 reply; 8+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-08-11  1:00 UTC (permalink / raw)
  To: Mou Tong; +Cc: 65190@debbugs.gnu.org

Mou Tong <mou.tong@outlook.com> writes:

> Ah, sorry.  I forgot to run `./autogen.sh` again to generate a new
> `configure`.
>
> After change `configure.ac` like you said before, and exec
> `./autogen.sh` to generate a new `configure`. Running `./configure`
> works as I expected.
>
> Po Lu <luangruo@yahoo.com> writes:
>
>> Does the problem resolve itself if you change:
>>
>>     [AS_IF([test "x$XCONFIGURE" != "xandroid" \
>>             && test "$with_android" = "no"],
>>       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
>>
>>
>> to read
>>
>>     [AS_IF([test "x$XCONFIGURE" != "xandroid"],
>>       [(movemail --version) >/dev/null 2>&1 || with_mailutils=no],
>>
>> ?
>
> This had solved the problem ;)

I fixed this differently.  Would you please pull from master and ack?





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-11  1:00         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-08-11  1:37           ` Mou Tong
  2023-08-11  1:51             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 8+ messages in thread
From: Mou Tong @ 2023-08-11  1:37 UTC (permalink / raw)
  To: Po Lu; +Cc: 65190@debbugs.gnu.org

Po Lu <luangruo@yahoo.com> writes:

> I fixed this differently.  Would you please pull from master and ack?

I pulled the latest commit cc6fed326aa491f89cad5cf59f4a8fb7b2863011
and it worked fine on my machine :)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS
  2023-08-11  1:37           ` Mou Tong
@ 2023-08-11  1:51             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 0 replies; 8+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-08-11  1:51 UTC (permalink / raw)
  To: Mou Tong; +Cc: 65190@debbugs.gnu.org

Mou Tong <mou.tong@outlook.com> writes:

> Po Lu <luangruo@yahoo.com> writes:
>
>> I fixed this differently.  Would you please pull from master and ack?
>
> I pulled the latest commit cc6fed326aa491f89cad5cf59f4a8fb7b2863011
> and it worked fine on my machine :)

Thanks, closing.





^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2023-08-11  1:51 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-10  2:06 bug#65190: 30.0.50; Emacs doesn't build with `--with-mailutils` by default when mailutils is installed on macOS Mou Tong
2023-08-10  2:37 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-08-10  3:10   ` Mou Tong
2023-08-10  4:47     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-08-10  6:04       ` Mou Tong
2023-08-11  1:00         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-08-11  1:37           ` Mou Tong
2023-08-11  1:51             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).