copyDirectory fails on read-only files.
Nobody has claimed this yet.
- Dominant language
- Macaulay2
- Stars
- 435
- Forks
- 297
- Avg merge
- 4d 20h
- Merged PRs (30d)
- 11
Description
Since the Style package has a Makefile created from its Makefile.in in its auxiliary files directory,
"installPackage" was failing on it, because "copyDirectory" has a problem.
Perhaps add an option to copyDirectory so it forcibly replaces read-only files.
Macaulay2, version 1.9
with packages: ConwayPolynomials, Elimination, IntegralClosure, LLLBases, PrimaryDecomposition, ReesAlgebra, TangentCone
i1 : copyDirectory("a", "b")
i2 : copyDirectory("a", "b")
stdio:2:1:(3): error: opening output file "b/foo" failed: Permission denied
i3 : run "ls -l a"
total 0
-r--r--r-- 1 dan dan 0 May 5 14:53 foo
o3 = 0
i4 : run "ls -l b"
total 0
-r--r--r-- 1 dan dan 0 May 5 14:53 foo
o4 = 0
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the copyDirectory and installPackage entry points, using the read-only-file example in the issue to reproduce the failure. Check that copying into an existing destination can replace read-only files and that installPackage no longer fails for the Style package's generated Makefile; verify the behavior with a focused test or reproduction.
Written by the indexing model from the issue text.
Assessment
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100