Skip to content

Commit

Permalink
Merge pull request #28 from Vanilla-OS/fix/includes-not-being-include…
Browse files Browse the repository at this point in the history
…d-in-recipe

fix: included modules not being included in the final recipe
  • Loading branch information
axtloss authored Apr 6, 2024
2 parents 7fe1054 + cdd0f2f commit 1242bb4
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion core/loader.go
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ func LoadRecipe(path string) (*api.Recipe, error) {
}
}

for _, stage := range recipe.Stages {
for i, stage := range recipe.Stages {
// here we check if the extra Adds path exists
for src := range stage.Adds {
fullPath := filepath.Join(filepath.Dir(recipePath), src)
Expand Down Expand Up @@ -153,6 +153,7 @@ func LoadRecipe(path string) (*api.Recipe, error) {
} else if followsGhPattern(include) {
// if the include follows the github pattern, we need to
// download the recipe from the github repository
fmt.Printf("Downloading recipe from %s\n", include)
modulePath, err = downloadGhRecipe(include)
if err != nil {
return nil, err
Expand All @@ -176,6 +177,7 @@ func LoadRecipe(path string) (*api.Recipe, error) {
}

stage.Modules = newRecipeModules
recipe.Stages[i] = stage
}

return recipe, nil
Expand Down

0 comments on commit 1242bb4

Please sign in to comment.