Ticket #268 (closed defect: wontfix)

Opened 2 years ago

Last modified 2 years ago

wrong mkdir_p case in trunk/src/Makefile.am

Reported by: orbisvicis Owned by: lennart
Milestone: Component: build-system
Keywords: Cc:

Description

mkdir_p not MKDIR_P to match automake. Issue not present in some branches anyway

Attachments

MakefilePatch (406 bytes) - added by orbisvicis 2 years ago.
config.log (228.6 kB) - added by orbisvicis 2 years ago.
configureOutput.tar.gz (142.3 kB) - added by orbisvicis 2 years ago.

Change History

Changed 2 years ago by orbisvicis

Changed 2 years ago by lennart

  • status changed from new to closed
  • resolution set to wontfix

MKDIR_P is part of autoconf 2.60. Please upgrade your autoconf.

Changed 2 years ago by orbisvicis

  • status changed from closed to reopened
  • resolution wontfix deleted
  • severity changed from trivial to minor

Yes you may be right but the issue still exists: (http://pastebin.com/m3cd99740)

automake --version automake (GNU automake) 1.9.6 ...

autoconf --version autoconf (GNU Autoconf) 2.61 ...

make --version GNU Make 3.81 ...

(and yes i do have other autotool versions installed on the system, but my defaults are the newest releases)

In the Makefile.in file, line 1676 mkdir_p = @mkdir_p@ (Is this normal? autoconf uses MKDIR_P but automake still uses mkdir_p, and automake created Makefile.in)

In the Makefile, line 341 mkdir_p = mkdir -p -- (this is clearly an issue)

I would say it is caused by autogen.sh/bootstrap.sh except: ..... + P=aclocal-1.9 + shift 2 + aclocal-1.9 ..... P=autoconf + shift 2 + autoconf -Wall ..... P=automake-1.9 + shift 2 + automake-1.9 --copy --foreign --add-missing .....

It is probably not caused by the configure script b/c when commenting out the configure section out of bootstrap.sh, the src/Makefile.am and src/Makefile.in still have a discrepancy in case. (Nor does ./configure even modify Makefile.in and Makefile.am) But config.status creates the Makefile... maybe MASH compatibility and AS_MKDIR_P ?

Changed 2 years ago by orbisvicis

Changed 2 years ago by orbisvicis

Changed 2 years ago by lennart

  • status changed from reopened to closed
  • resolution set to wontfix

I just follow what the automake docs suggest:

http://sources.redhat.com/automake/automake.html#Obsolete-macros

If that's not correct with your version of automake, please file a bug to automake.

I now bumped the required automake version in configure.ac to make clear that this might not work on old automake versions.

Note: See TracTickets for help on using tickets.