Opened 4 years ago

Closed 2 years ago

#26840 closed defect (invalid)

Copying __pycache__ fails during clean build

Reported by: Markus Wageringel Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords:
Cc: Merged in:
Authors: Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


During a clean build of Sage 8.5.beta6, I experience the error below on OS X 10.13.6. It fails to copy the directory src/bin/__pycache__ to local/bin.

This is fixed by adding -R to the corresponding cp command.

[sagelib-8.5.beta6] real        0m12.124s
[sagelib-8.5.beta6] user        0m7.640s
[sagelib-8.5.beta6] sys 0m3.659s
cp /Applications/SageMath/src/bin/__pycache__ /Applications/SageMath/local/bin/__pycache__
cp: /Applications/SageMath/src/bin/__pycache__ is a directory (not copied).
make[3]: *** [/Applications/SageMath/local/bin/__pycache__] Error 1
make[2]: *** [all-build] Error 2

real    0m12.705s
user    0m8.042s
sys     0m3.826s
Error building Sage.

Previously reported here:!topic/sage-devel/KzIbSW1J8KU

Change History (5)

comment:1 Changed 4 years ago by Markus Wageringel

Authors: Markus Wageringel
Status: newneeds_review

comment:2 Changed 3 years ago by Markus Wageringel

Milestone: sage-8.5sage-8.8

I am still getting build failures because of this issue, running make distclean && make. Does no one else have this problem?

comment:3 Changed 3 years ago by Markus Wageringel

Authors: Markus Wageringel
Branch: u/gh-mwageringel/build/cp_pycache
Commit: 42bcc448c2fe19939699dfb6744e08c7e8eba659
Milestone: sage-8.8sage-duplicate/invalid/wontfix

I removed the branch. This problem can be avoided by removing all __pycache__ folders (which happened to be empty) from the directory src/sage/ and then running make distclean to erase any mention of __pycache__ from the autogenerated Makefiles. I am not sure where those folders were coming from, but it does not seem to be a common problem.

This ticket can be closed.

comment:4 Changed 2 years ago by Matthias Köppe

Reviewers: Matthias Koeppe
Status: needs_reviewpositive_review

comment:5 Changed 2 years ago by Frédéric Chapoton

Resolution: invalid
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.