Skip to content

Conversation

@neelpatel05
Copy link
Contributor

No description provided.

@skinkie
Copy link
Member

skinkie commented Dec 9, 2019

Could you elaborate why this is required?

@neelpatel05
Copy link
Contributor Author

./autogen.sh --prefix=/usr --sysconfdir=/etc --localstatedir=/var

While installing the Cherokee server using the above command to generate make file, the script stops executing and throws an error in "Automake" because the underlying files are in the subdirectory. Thus, this change in Configure.ac file resolves the installation error

@skinkie
Copy link
Member

skinkie commented Dec 9, 2019

And just to help me this to reproduce (I have not seen it before) which version of automake causes this?

@neelpatel05
Copy link
Contributor Author

The version of Automake that I used is 1.16.1

@skinkie
Copy link
Member

skinkie commented Dec 15, 2019

I cannot reproduce this issue with 1.15. But I'll follow up.

skinkie added a commit that referenced this pull request Oct 22, 2021
With a recent upgrade of a development system I have noticed the issues from #1249 and #1239. While both did not resolve the problems I encountered I was able to resolve the issues in the following way.

Starting from the work of @rdratlos I noticed that AM_INIT_AUTOMAKE was defined twice, this alone caused the direct breaking of autogen.sh with recent versions of automake and autoconf. Removing the second instance resolved the issue. While @neelpatel05 suggested to use AM_INIT_AUTOMAKE with "foreign subdir-objects" I have still not reproduced that using automake 1.16.5. Hence I have ignored this suggestion, but if someone can show how it breaks without it I'll obviously include it.

autoupdate-2.71 failed apply an update and terminated processing at an old AC_TRY_COMPILE code block of "Check of func and co.." I manually rewrote the AC_COMPILE_IFELSE. Applying autoupdate-2.71 to the rest of the m4 files in the repository allowed me to clean up all the remaining warnings.

Co-authored-by: Thomas Reim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants