Don't fail in stage subcmds when --workspace is undefined but --sta…

…ge-store is set (#180)

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+## 0.7.0-alpha.3 - 2024-04-13
+
+### Fixed
+
+- (cli) Subcommands under `stage` no longer require the workspace to be set via `--workspace` or `FORKLIFT_WORKSPACE` (which defaults to `$HOME`, which may be unset in systemd system services) if the path to the stage store is explicitly set via `--stage-store` or `FORKLIFT_STAGE_STORE`.
+
 ## 0.7.0-alpha.2 - 2024-04-13
 
 ### Added

cmd/forklift/stage/store.go

@@ -65,13 +65,20 @@ func loadNextBundle(
 	return bundle, store, nil
 }
 
-func getStageStore(wpath, sspath string, versions Versions) (*forklift.FSStageStore, error) {
-	workspace, err := forklift.LoadWorkspace(wpath)
-	if err != nil {
-		return nil, err
+func getStageStore(
+	wpath, sspath string, versions Versions,
+) (store *forklift.FSStageStore, err error) {
+	var workspace *forklift.FSWorkspace
+	if sspath == "" {
+		workspace, err = forklift.LoadWorkspace(wpath)
+		if err != nil {
+			return nil, errors.Wrap(
+				err, "couldn't load workspace to load the stage store, since no explicit path was "+
+					"provided for the stage store",
+			)
+		}
 	}
-	store, err := fcli.GetStageStore(workspace, sspath, versions.NewStageStore)
-	if err != nil {
+	if store, err = fcli.GetStageStore(workspace, sspath, versions.NewStageStore); err != nil {
 		return nil, err
 	}
 	return store, nil