Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix configure messages "rm: conftest.dir: is a directory" #29489

Closed
mkoeppe opened this issue Apr 9, 2020 · 8 comments
Closed

Fix configure messages "rm: conftest.dir: is a directory" #29489

mkoeppe opened this issue Apr 9, 2020 · 8 comments
Labels
c: build: configure p: minor / 4 t: bug
Milestone
sage-9.1

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Apr 9, 2020

Some autoconf macro is trying to use rm -f when it should be using rm -rf, leading to these harmless error messages:

checking for ZMQ version >= 4.2.5... yes
rm: conftest.dir: is a directory
rm: conftest_venv: is a directory
configure: will use system package and not install SPKG zeromq

CC: @dimpase @orlitzky

Component: build: configure

Author: Michael Orlitzky

Branch/Commit: d208f8c

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/29489
@mkoeppe mkoeppe added this to the sage-9.1 milestone Apr 9, 2020
@orlitzky
Copy link
Contributor

New commits:

d208f8cTrac #29489: clean up conftest.dir and conftest_venv after use.

@orlitzky
Copy link
Contributor

Commit: d208f8c

@orlitzky
Copy link
Contributor

Branch: u/mjo/ticket/29489

@orlitzky
Copy link
Contributor

Author: Michael Orlitzky

@dimpase
Copy link
Member

dimpase commented Apr 18, 2020

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Apr 18, 2020

comment:2

lgtm, thanks!

@mkoeppe
Copy link
Member Author

mkoeppe commented Apr 18, 2020

comment:3

Thanks for fixing this!

@vbraun
Copy link
Member

vbraun commented Apr 22, 2020

Changed branch from u/mjo/ticket/29489 to d208f8c

@vbraun vbraun closed this as completed in f00937e Apr 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: build: configure p: minor / 4 t: bug
Projects
None yet
Milestone
sage-9.1
Development

No branches or pull requests
4 participants
@vbraun @dimpase @orlitzky @mkoeppe