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

Feat: Allow multiple sources in plugins #126

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Feat: Allow multiple sources in plugins #126

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion api/download.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ func DownloadGitSource(downloadPath string, source Source, moduleName string) er
}

// Default to latest commit
if source.Commit == "" || source.Commit == "latest" {
if len(strings.TrimSpace(source.Commit)) == 0 || strings.EqualFold(source.Commit, "latest") {
source.Commit, err = gitGetLatestCommit(source.Branch, dest)
if err != nil {
return fmt.Errorf("could not get latest commit: %s", err.Error())
Expand Down
23 changes: 22 additions & 1 deletion finalize-plugins/systemd-repart.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ type SystemdRepart struct {
Size string `json:"size"`
Seed string `json:"seed"`
Split bool `json:"split"`
Empty string `json:"empty"`
Root string `json:"root"`
DeferPartitions []string `json:"defer_partitions"`
}

Expand All @@ -42,6 +44,15 @@ func PluginScope() int32 { // int32 is defined as GoInt32 in cgo which is the sa
return api.IMAGENAME | api.FS | api.RECIPE
}

// Replace placeholders in the path with actual values from ScopeData
// $PROJROOT -> Recipe.ParentPath
// $FSROOT -> FS
func parsePath(path string, data *api.ScopeData) string {
path = strings.ReplaceAll(path, "$PROJROOT", data.Recipe.ParentPath)
path = strings.ReplaceAll(path, "$FSROOT", data.FS)
return path
}

// Finalize the build by executing systemd-repart with the provided configuration
// to generate and apply partitioning specifications and output results
//
Expand Down Expand Up @@ -69,9 +80,19 @@ func FinalizeBuild(moduleInterface *C.char, extraData *C.char) *C.char {
module.Json = "off"
}

if len(strings.TrimSpace(module.Empty)) == 0 {
module.Empty = "create"
}

if len(strings.TrimSpace(module.Root)) == 0 {
module.Root = data.FS
} else {
module.Root = parsePath(module.Root, data)
}

args := []string{
"--definitions=definitions",
"--empty=create",
fmt.Sprintf("--empty=%s", module.Empty),
fmt.Sprintf("--size=%s", module.Size),
"--dry-run=no",
"--discard=no",
Expand Down
2 changes: 1 addition & 1 deletion plugins/cmake.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ func BuildModule(moduleInterface *C.char, recipeInterface *C.char) *C.char {
}

cmd := fmt.Sprintf(
"cd /sources/%s && mkdir -p build && cd build && cmake ..%s && make",
"cd /sources/%s && mkdir -p build && cd build && cmake ../%s && make",
filepath.Join(recipe.SourcesPath, api.GetSourcePath(module.Source, module.Name)),
buildFlags,
)
Expand Down
30 changes: 16 additions & 14 deletions plugins/make.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,12 @@ import "strings"

// Configuration for building a project using Make
type MakeModule struct {
Name string `json:"name"`
Type string `json:"type"`
BuildCommand string `json:"buildcommand"`
InstallCommand string `json:"installcommand"`
IntermediateSteps []string `json:"intermediatesteps"`
Source api.Source
Name string `json:"name"`
Type string `json:"type"`
BuildCommand string `json:"buildcommand"`
InstallCommand string `json:"installcommand"`
IntermediateSteps []string `json:"intermediatesteps"`
Sources []api.Source `json:"sources"`
}

// Provide plugin information as a JSON string
Expand Down Expand Up @@ -50,13 +50,15 @@ func BuildModule(moduleInterface *C.char, recipeInterface *C.char) *C.char {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}

err = api.DownloadSource(recipe, module.Source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
err = api.MoveSource(recipe.DownloadsPath, recipe.SourcesPath, module.Source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
for _, source := range module.Sources {
err = api.DownloadSource(recipe, source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
err = api.MoveSource(recipe.DownloadsPath, recipe.SourcesPath, source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
}

buildCommand := "make"
Expand All @@ -75,7 +77,7 @@ func BuildModule(moduleInterface *C.char, recipeInterface *C.char) *C.char {
intermediateSteps = " && " + strings.Join(module.IntermediateSteps, " && ") + " && "
}

cmd := "cd /sources/" + api.GetSourcePath(module.Source, module.Name) + " && " + buildCommand + intermediateSteps + installCommand
cmd := "cd /sources/" + api.GetSourcePath(module.Sources[0], module.Name) + " && " + buildCommand + intermediateSteps + installCommand
return C.CString(cmd)
}

Expand Down
34 changes: 21 additions & 13 deletions plugins/meson.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,14 @@ import (

"github.com/vanilla-os/vib/api"
)
import "crypto/sha1"

// Configuration for building a Meson project
type MesonModule struct {
Name string
Type string
BuildFlags []string `json:"buildflags"`
Source api.Source
BuildFlags []string `json:"buildflags"`
Sources []api.Source `json"sources"`
}

// Provide plugin information as a JSON string
Expand Down Expand Up @@ -47,21 +48,28 @@ func BuildModule(moduleInterface *C.char, recipeInterface *C.char) *C.char {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}

err = api.DownloadSource(recipe, module.Source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
err = api.MoveSource(recipe.DownloadsPath, recipe.SourcesPath, module.Source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
for _, source := range module.Sources {
err = api.DownloadSource(recipe, source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
err = api.MoveSource(recipe.DownloadsPath, recipe.SourcesPath, source, module.Name)
if err != nil {
return C.CString(fmt.Sprintf("ERROR: %s", err.Error()))
}
}

// Since the downloaded source goes through checksum verification already
// it is safe to simply use the specified checksum from the module definition
tmpDir := fmt.Sprintf("/tmp/%s-%s", module.Source.Checksum, module.Name)
var tmpDir string
if strings.EqualFold(module.Sources[0].Type, "git") == true {
tmpDir = fmt.Sprintf("/tmp/%s-%s", module.Sources[0].Commit, module.Name)
} else if module.Sources[0].Type == "tar" || module.Sources[0].Type == "local" {
tmpDir = fmt.Sprintf("/tmp/%s-%s", module.Sources[0].Checksum, module.Name)
} else {
tmpDir = fmt.Sprintf("/tmp/%s-%s", sha1.Sum([]byte(module.Sources[0].URL)), module.Name)
}
cmd := fmt.Sprintf(
"cd /sources/%s && meson %s %s && ninja -C %s && ninja -C %s install",
api.GetSourcePath(module.Source, module.Name),
api.GetSourcePath(module.Sources[0], module.Name),
strings.Join(module.BuildFlags, " "),
tmpDir,
tmpDir,
Expand Down
Loading