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.

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

P7: Instance of Diskstore doesn't understand #isWritable #71

Open
Bajger opened this issue Apr 25, 2022 · 0 comments
Open

P7: Instance of Diskstore doesn't understand #isWritable #71

Bajger opened this issue Apr 25, 2022 · 0 comments

Comments

@Bajger
Copy link

Bajger commented Apr 25, 2022

Bug description
In Exercism Pharo track is OSSubprocess project used. However it fails on P7 (backward compatibility is handled there).
During initialization of OSVMProcess class, is OldStandardFileStream>>open:forWrite: sent, which causes a DNU message.

To Reproduce
Steps to reproduce the behavior:

  1. Download P7 official release
  2. Run MC command:
Metacello new
 	baseline: 'OSSubprocess';
 	repository: 'github://pharo-contributions/OSSubprocess:v1.3.0/repository';
	load.
  1. See error: "Instance of FileSystem did not understand #isWritable"

Expected behavior
Package is successfully loaded

Screenshots
image

Version information:

  • OS: macOS Monterey (12.2.1)
  • Pharo Version P7-64bit stable

Expected development cost

  1. Add P7 compatibility package that would be loaded prior OSSubprocess package.
  2. Change implementation of OldStandardFileStream>>open:forWrite:
    to use fileName asFileReference isWriteable instead of: FileSystem disk isWritable not that causes DNU.
  3. Add a change to compatibility package.

Additional context
P7 is used by some of the users of Exercism Pharo track.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant