Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

[DNM] [WIP] Library #652

Merged
merged 19 commits into from
Aug 27, 2021
Merged

[DNM] [WIP] Library #652

merged 19 commits into from
Aug 27, 2021

Conversation

imkiva
Copy link
Member

@imkiva imkiva commented Aug 22, 2021 •
edited

This PR implements a library system for Aya.

TODOs:

  • library directory structure and docs
  • save compiled core
  • reuse compiled core to speed up compilation
  • make recompile decision with a file dependency graph
  • move some code to package manager (done in another PR)

If we are going to make package manager a Java project, I propose we could move all the code from cli to package manager

@ice1000 ice1000 added this to the v0.9 milestone Aug 22, 2021
@ice1000 ice1000 added the libraries A system of libraries label Aug 22, 2021
@ice1000 ice1000 changed the base branch from main to graal August 22, 2021 23:52
@ice1000 ice1000 changed the base branch from graal to main August 22, 2021 23:52
@ice1000
Copy link
Member

ice1000 commented Aug 22, 2021

Some changes are cherry picked as #653

This was linked to issues Aug 23, 2021
Library system #413
Closed
Random thoughts on library system and module lookup #491
Closed
@ice1000 ice1000 mentioned this pull request Aug 23, 2021
Merged
@ice1000 ice1000 force-pushed the library branch 7 times, most recently from b605d07 to 83936e3 Compare August 26, 2021 20:01
@ice1000 ice1000 self-assigned this Aug 27, 2021
ice1000
ice1000 approved these changes Aug 27, 2021
View reviewed changes
Copy link
Member

@ice1000 ice1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@imkiva proposed to merge this early. Let's do this.

@ice1000
Copy link
Member

ice1000 commented Aug 27, 2021

I've done the review. @imkiva please create new issue with the leftovers.

@ice1000 ice1000 merged commit 6a56824 into main Aug 27, 2021
@ice1000 ice1000 deleted the library branch August 27, 2021 05:34
@ice1000 ice1000 mentioned this pull request Sep 28, 2021
Closed
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ice1000 ice1000 ice1000 approved these changes

Assignees

@ice1000 ice1000

Labels
libraries A system of libraries
Projects
None yet
Milestone
v0.8.3
Development

Successfully merging this pull request may close these issues.

Random thoughts on library system and module lookup Library system
2 participants
@imkiva @ice1000