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.

Sign up for GitHub

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

Jump to bottom

Infinitive loop in #prerequisites #81

Closed
marianopeck opened this issue Jun 25, 2021 · 1 comment
Closed

Infinitive loop in #prerequisites #81

marianopeck opened this issue Jun 25, 2021 · 1 comment
Assignees
@eMaringolo
Labels
enhancement New feature or request importing Issues related with importing from Tonel format
Milestone
1.3

Comments

@marianopeck
Copy link
Collaborator

marianopeck commented Jun 25, 2021 •
edited
Loading

prerequisites -> computePrerequisites -> computeImplicitPrerequisites -> allPrerequisites -> prerequisites

Loop.

This is reproduced by:

repoPath := (CfsPath named: 'z:\Common\Development\git\Grease\').
reader := TonelReader new readFrom: repoPath.
loader := TonelLoader on: reader.
loader beUnattended; doNotCreateVersions.
loader mapTagsToSubapplications.
loader namingStrategy appSuffix: 'App'.
loader applications.
loader loadApplicationForPackageNamed: 'Grease-Tests-Core'

Attached is the walkback.
walkback.txt
eMaringolo added a commit that referenced this issue Jun 25, 2021
@eMaringolo
Changes for #81
a2b554a
@marianopeck
Copy link
Collaborator Author

Fixed.

eMaringolo added a commit that referenced this issue Jun 25, 2021
@eMaringolo
Adds prerequistesFilter to fix #81
203e4b8
@eMaringolo eMaringolo self-assigned this Jun 25, 2021
@eMaringolo eMaringolo added enhancement New feature or request importing Issues related with importing from Tonel format labels Jun 25, 2021
@eMaringolo eMaringolo added this to the 1.3 milestone Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eMaringolo eMaringolo

Labels
enhancement New feature or request importing Issues related with importing from Tonel format
Projects
None yet
Milestone
1.3
Development

No branches or pull requests
2 participants
@marianopeck @eMaringolo