-
Notifications
You must be signed in to change notification settings - Fork 291
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
Removing AC_CHECK_FILE from slepc.m4 #17
Conversation
AC_CHECK_FILE does not play nicely with cross-compiling.
libMesh - C++ Finite Element Library » libmesh #83 SUCCESS |
Sure - if not tonight first thing tomorrow. -Ben On Dec 18, 2012, at 4:21 PM, "jwpeterson" <[email protected]mailto:[email protected]> wrote: Can someone who actually uses SLEPc (or has a buildbot configuration for it) try out this branch and let me know if it still works? Thanks... You can merge this Pull Request by running: git pull https://github.com/jwpeterson/libmesh slepc_check_header Or view, comment on, or merge it at: Commit Summary
File Changes
Patch Links — |
Thanks. Forgot to mention: new configure script is not included in the patch, so you'll have to bootstrap to try out the changes. |
This works for me. BTW, what's the cleanest way to get back to a pristine master after I pull a single file like this? I can check out the original again from master, but that still shows me with local mods... -Ben |
Hmm if you pulled it down into master, you can git reset --hard HEAD~1 To get rid of it... John On Dec 18, 2012, at 5:19 PM, "Benjamin S. Kirk" [email protected] wrote:
|
That worked, thanks! If you want to merge these that's fine with me - it'll give my builbot a chance to see them in master and I can fix any issues tomorrow... |
Ok will do that after the kids are in bed. John On Dec 18, 2012, at 6:28 PM, "Benjamin S. Kirk" [email protected] wrote:
|
Removing AC_CHECK_FILE from slepc.m4
Can someone who actually uses SLEPc (or has a buildbot configuration for it) try out this branch and let me know if it still works? Thanks...