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

Bootstrap is really hard to update #4269

Open
jecisc opened this issue Aug 10, 2019 · 3 comments

Comments

@jecisc
Copy link
Member

commented Aug 10, 2019

Currently, I did not found how to update the bootstrap code within Pharo.

When you add the pharo repository it shows the sources in src/ but not bootstrap/src/.

I tried to edit the pharo repository in Iceberg to update the location to bootstrap/src/ and the format to tonel, but it failed to display any package. I guess it misses a property file.

I tried to add the repository as a FileTree repository in Monticello but could not load the code. I got a lot of errors.

When I changed the bootstrap I had to do it by hand in a file editor, which is not optimal.

I see multiple things that we can improve:

  • We need to fix Monticello (else we lose every project that was not migrated or whose owner do not want to put on github)
  • We should fix the properties files of the bootstrap
  • Instead of the previous proposition, we can convert the bootstrap to Tonel
  • We should manage multiple source directories in Iceberg (but this one is a pretty big feature)
  • We can extract the bootstrap to its own repository instead of the previous proposition
  • Maybe someone else has other ideas?
@guillep

This comment has been minimized.

Copy link
Member

commented Aug 10, 2019

  • We need to fix Monticello (else we lose every project that was not migrated or whose owner do not want to put on github)

How is monticello broken? I've used Monticello to load code yesterday from squeaksource and smalltalkhub and it worked just fine.
Also this does not seem a bootstrap problem or maybe I'm missunderstanding. In case there is an issue, can you add a separate issue for this?

  • We should fix the properties files of the bootstrap
  • Instead of the previous proposition, we can convert the bootstrap to Tonel

Both are true and should be simple enough to do. Probably to discuss with @carolahp.

  • We should manage multiple source directories in Iceberg (but this one is a pretty big feature)

If you have enough money to put on the table we can discuss this :)

  • We can extract the bootstrap to its own repository instead of the previous proposition

No, this will be a super mess of dependencies. The code that bootstraps PharoX is together with PharoX.
Having something in the middle of both would be better.

True, there are many points of enhacements.
@carolahp has updated the bootstrap to run on Pharo7, I don't know in which state is this.
Also with the vmmaker source code in github this could be further simplified.

However, I have to ask, have you tried following the instructions in the readme here?
https://github.com/pharo-project/pharo

GitHub
The Sources for Pharo. Contribute to pharo-project/pharo development by creating an account on GitHub.
@jecisc

This comment has been minimized.

Copy link
Member Author

commented Aug 10, 2019

How is monticello broken?

If you add the pharo bootstrap to Monticello as a Filetree repository and try to load a package you get error after error.

However, I have to ask, have you tried following the instructions in the readme here?

The README explains how to bootstrap a Pharo, not how to update the Pharo code of the bootstrap. Or maybe I missed this section.

@noha

This comment has been minimized.

Copy link
Member

commented Aug 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.