-
Notifications
You must be signed in to change notification settings - Fork 114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configure troubles #166
Comments
Hi and thanks for posting. Your post is timely since I'm in the middle of improving the documentation, where I can take this into account. Both our docs and releases are a bit dated, which I'll fix soon. About your points:
|
I've made a note of the "official" release. That one includes a configure script and it builds without a hitch. Bootstrap on the clone gives me:
|
The best branch to clone is prev3-develop. It's modern and well-tested. It will have the proper branch configured for the submodule sc as well. Would that work better for you? |
I've bumped the master, develop and prev3-develop branches to the same state. For modern yet conservative code, use the master branch. The latest will be on the development branches. I've so far not been happy with the github releases. Please use the latest tarball from p4est.org. Anything left to address version wise? |
My install script reports the following:
|
Autoreconf is not working right, that's why in ./bootstrap I've been hardcoding the individual steps. Could you just call |
Bump and also relating this to #217. |
It's been a while. I'm reading through this....
|
What on earth? 2.8.5.tar.gz is not in gzip format it seems. It contains
|
Problems with bootstrap approach:
|
It continues:
|
Weird. The github way of releases is not cool. Please use the .tar.gz files linked from p4est.org. |
We try to do x.y.z. Just there were some updates after the fact that made us release p4est-2.8.5.5-9ddbb.tar.gz. Does that one work for you? Right now I cannot reproduce your script output about the pkgconfig_DATA. It runs for me but I'll double check. |
The 2.8.5 came from the p4est.org page. Specifically that pointed me to github for older tarballs. |
Hi, I have seen a similar issue on macOS 12.6.9.
I always install p4est through petsc which worked out smoothly. But I have seen this issue very recently. I believe petsc used to point to an old p4est: Any hint would be helpful. Thanks. |
If you just cloned p4est from github, there will be a mostly empty subdirectory sc. |
No. I tried to install p4est alone through a clone and it has the same error. I did run I had some system upgrade recently, and had to re-install libtool/autconf through brew. I am not sure if that is related. The previous successful installed p4est on the same laptop was on eb24772 |
Thanks for the research! I'd like the pkgconfig logic supplied by autoconf and automake in the most vanilla way possible, that is, consistent with the best practices from available documentation and simplified to the minimum. There should only be a couple macros and variables to be examined and compared with the documented state of the art. Remove all that seems too out of the ordinary. Would you like to look into it? |
Sure, happy to help if you can give me some hints. What you said makes sense. You mean I should have a look at the most recent automake/autconf doc and figure out the redundant ones in your current |
Thanks! Yes, whatever you think may need updating in the Makefile.am files and under config/. |
We have released v2.8.6. The authoritative tarball is p4est-2.8.6.tar.gz linked from our pages and attached to the github release. Please do not use github's auto-generated zip and tar files, they are missing the sc submodule and more. Would this file work as expected for y'all? |
I'm leaning towards closing this as resolved. Any further issues? |
My install script now has
as the parametrized location of the archive. If that is correct, you can close this ticket afaic. |
My install script now has
```
https://github.com/cburstedde/p4est/releases/download/v${PACKAGEVERSION}/p4est-${PACKAGEVERSION}.tar.gz
```
as the parametrized location of the archive. If that is correct, you can close this ticket afaic.
Thanks for the feedback. This is fine as long as github doesn't change
their convention.
|
Trying to install 2.3.3 (after trying 2.8)
The readme states "Our official tarballs contain the configure script" but I'm not finding any official releases under https://github.com/cburstedde/p4est/releases
Running autoconf gives
According to a 20 year post I need to run
aclocal
https://lists.gnu.org/archive/html/autoconf/2004-09/msg00215.htmlOk, that seems to have helped.
3. Now I'm at
The text was updated successfully, but these errors were encountered: