#31002 closed enhancement (fixed)

Specify bdist temp folder for WSL compatibility

Reported by: gh-tobiasdiez Owned by:
Priority: critical Milestone: sage-9.3
Component: build Keywords: sd111
Cc: mkoeppe Merged in:
Authors: Tobias Diez, Matthias Koeppe Reviewers: Matthias Koeppe, Tobias Diez
Report Upstream: N/A Work issues:
Branch: fb10429 (Commits, GitHub, GitLab) Commit: fb10429b974d649741c67077ed50a00815880932
Dependencies: Stopgaps:

Status badges

Description (last modified by slelievre)

On WSL, bdist_wheel sometimes encounters strange permission errors (when the source folder is a Windows folder mounted in WSL), see https://github.com/pypa/packaging-problems/issues/258. In the linked issue, a solution is described by changing the permissions for authenticated users, but it didn't work for me. Thus I've implemented the workaround mentioned in https://github.com/pypa/packaging-problems/issues/258#issuecomment-548148041 and specified an explicit bdist directory in the Linux system.

Change History (18)

comment:1 Changed 21 months ago by gh-tobiasdiez

  • Status changed from new to needs_review

comment:2 Changed 21 months ago by mkoeppe

Thanks for tracking down this problem.

I don't think using ~/temp/ is a good idea, though. I think a better solution would be to use the command mktemp -d to create a directory in the system directory for temporary files.

comment:3 Changed 21 months ago by git

  • Commit changed from a8e0364dc6ff16a799b64670e75f05a4dfac3b1e to a3077c71f488ddb2b6fe4058cbbc981d02d7ba95

Branch pushed to git repo; I updated commit sha1. New commits:

a3077c7Use mktemp

comment:4 Changed 21 months ago by gh-tobiasdiez

ok done

comment:5 Changed 21 months ago by mkoeppe

The temporary directory should probably be removed afterwards...

Probably best to create a new function sdh_setup_bdist_wheel in build/bin/sage-dist-helpers for that

comment:6 Changed 21 months ago by gh-tobiasdiez

comment:7 follow-ups: Changed 21 months ago by mkoeppe

But the directory is not removed as far as I can see

comment:8 in reply to: ↑ 7 ; follow-up: Changed 21 months ago by gh-tobiasdiez

Replying to mkoeppe:

But the directory is not removed as far as I can see

Probably. If you think this is a problem, I would like to ask you to implement your suggestion with the sage-dist-helpers. My shell knowledge is sadly not good enough for this.

comment:9 Changed 21 months ago by mkoeppe

  • Status changed from needs_review to needs_work

OK, will do

comment:10 Changed 21 months ago by git

  • Commit changed from a3077c71f488ddb2b6fe4058cbbc981d02d7ba95 to c5a14a396fd1f1cd2de0d6fd465e546b7ff0b33b

Branch pushed to git repo; I updated commit sha1. New commits:

c5a14a3build/bin/sage-dist-helpers (sdh_setup_bdist_wheel): New

comment:11 in reply to: ↑ 7 Changed 21 months ago by mkoeppe

Replying to mkoeppe:

But the directory is not removed as far as I can see

Actually I was wrong about this

comment:12 in reply to: ↑ 8 Changed 21 months ago by mkoeppe

Replying to gh-tobiasdiez:

I would like to ask you to implement your suggestion with the sage-dist-helpers.

Nevertheless I have implemented this function. Better to have it in one place.

I have only changed it in the pillow install script, where it seems to work.

Could you change the other ones?

comment:13 Changed 21 months ago by mkoeppe

  • Keywords sd111 added

Hoping we can make progress on this ticket this week - https://wiki.sagemath.org/days111

comment:14 Changed 21 months ago by git

  • Commit changed from c5a14a396fd1f1cd2de0d6fd465e546b7ff0b33b to fb10429b974d649741c67077ed50a00815880932

Branch pushed to git repo; I updated commit sha1. New commits:

fb10429build/pkgs/*/spkg-install.in: Use new function sdh_setup_bdist_wheel

comment:15 Changed 21 months ago by mkoeppe

  • Authors changed from Tobias Diez to Tobias Diez, Matthias Koeppe
  • Reviewers set to Matthias Koeppe, ...
  • Status changed from needs_work to needs_review

Changed the remaining ones.

comment:16 Changed 21 months ago by gh-tobiasdiez

  • Reviewers changed from Matthias Koeppe, ... to Matthias Koeppe, Tobias Diez
  • Status changed from needs_review to positive_review

Thanks! LGTM

comment:17 Changed 20 months ago by slelievre

  • Description modified (diff)

comment:18 Changed 20 months ago by vbraun

  • Branch changed from public/build/bdist_wsl to fb10429b974d649741c67077ed50a00815880932
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.