diff --git a/src/bpm/main.go b/src/bpm/main.go index fe427eb..d759555 100644 --- a/src/bpm/main.go +++ b/src/bpm/main.go @@ -89,7 +89,6 @@ func main() { currentFlagSet.BoolP("force", "f", false, "Bypass warnings during package installation") currentFlagSet.BoolP("yes", "y", false, "Enter 'yes' in all prompts") currentFlagSet.Bool("runtime", true, "Install all runtime dependencies") - currentFlagSet.BoolP("optional", "o", false, "Install all optional dependencies") currentFlagSet.String("installation-reason", "", "Specify the installation reason to use for the specified packages") currentFlagSet.BoolP("reinstall", "r", false, "Reinstall the specified packages") currentFlagSet.IntP("jobs", "j", bpmlib.CompilationBPMConfig.CompilationJobs, "Set the amount of concurrent processes to use for source package compilation") @@ -142,7 +141,6 @@ func main() { currentFlagSet.BoolP("yes", "y", false, "Enter 'yes' in all prompts") currentFlagSet.BoolP("no-sync", "n", false, "Do not sync databases") currentFlagSet.Bool("allow-downgrades", false, "Allow package downgrades") - currentFlagSet.BoolP("optional", "o", false, "Install all optional dependencies") currentFlagSet.BoolP("skip-checks", "s", false, "Skip the check function in source.sh scripts") currentFlagSet.IntP("jobs", "j", bpmlib.CompilationBPMConfig.CompilationJobs, "Set the amount of concurrent processes to use for source package compilation") setupFlagsAndHelp(currentFlagSet, fmt.Sprintf("bpm %s ", subcommand), "Update installed packages", os.Args[2:]) @@ -558,7 +556,6 @@ func installPackages() { force, _ := currentFlagSet.GetBool("force") yesAll, _ := currentFlagSet.GetBool("yes") installRuntime, _ := currentFlagSet.GetBool("runtime") - installOptional, _ := currentFlagSet.GetBool("optional") installationReason, _ := currentFlagSet.GetString("installation-reason") reinstallPackages, _ := currentFlagSet.GetBool("reinstall") skipChecks, _ := currentFlagSet.GetBool("skip-checks") @@ -620,7 +617,7 @@ func installPackages() { } // Create installation operation - operation, err := bpmlib.InstallPackages(rootDir, ir, reinstallPackages, installRuntime, installOptional, force, !skipChecks, verbose, packages...) + operation, err := bpmlib.InstallPackages(rootDir, ir, reinstallPackages, installRuntime, force, !skipChecks, verbose, packages...) if errors.As(err, &bpmlib.PackageNotFoundErr{}) || errors.As(err, &bpmlib.DependencyNotFoundErr{}) || errors.As(err, &bpmlib.PackageConflictErr{}) { log.Printf("Error: %s", err) exitCode = 1 @@ -705,7 +702,7 @@ func installPackages() { } // Show optional dependencies - if !installOptional && len(optionalDepends) != 0 { + if len(optionalDepends) != 0 { // List optional dependencies fmt.Println("The following optional dependenices have been discovered:") for dependant, depends := range optionalDepends { @@ -993,7 +990,6 @@ func updatePackages() { yesAll, _ := currentFlagSet.GetBool("yes") noSync, _ := currentFlagSet.GetBool("no-sync") allowDowngrades, _ := currentFlagSet.GetBool("allow-downgrades") - installOptional, _ := currentFlagSet.GetBool("optional") skipChecks, _ := currentFlagSet.GetBool("skip-checks") compilationJobs, _ := currentFlagSet.GetInt("jobs") @@ -1041,7 +1037,7 @@ func updatePackages() { } // Create update operation - operation, err := bpmlib.UpdatePackages(rootDir, !noSync, allowDowngrades, installOptional, force, !skipChecks, verbose) + operation, err := bpmlib.UpdatePackages(rootDir, !noSync, allowDowngrades, force, !skipChecks, verbose) if errors.As(err, &bpmlib.PackageNotFoundErr{}) || errors.As(err, &bpmlib.DependencyNotFoundErr{}) || errors.As(err, &bpmlib.PackageConflictErr{}) { log.Printf("Error: %s", err) exitCode = 1 @@ -1126,7 +1122,7 @@ func updatePackages() { } // Show optional dependencies - if !installOptional && len(optionalDepends) != 0 { + if len(optionalDepends) != 0 { // List optional dependencies fmt.Println("The following optional dependenices have been discovered:") for dependant, depends := range optionalDepends { diff --git a/src/bpmlib/dependencies.go b/src/bpmlib/dependencies.go index 174d4a4..21c2a24 100644 --- a/src/bpmlib/dependencies.go +++ b/src/bpmlib/dependencies.go @@ -100,7 +100,7 @@ type ResolvedPackage struct { InstallationReason InstallationReason } -func ResolveDependencies(pkgInfo *PackageInfo, resolvedVirtualPackages map[string]string, includeRuntimeDepends, includeOptionalDepends bool, rootDir string) (resolved []ResolvedPackage, unresolved []string) { +func ResolveDependencies(pkgInfo *PackageInfo, resolvedVirtualPackages map[string]string, includeRuntimeDepends bool, rootDir string) (resolved []ResolvedPackage, unresolved []string) { visited := make([]string, 0) var dfs func(resolvedPkg *PackageInfo) @@ -146,9 +146,6 @@ func ResolveDependencies(pkgInfo *PackageInfo, resolvedVirtualPackages map[strin checkDependencies(pkgInfo.MakeDepends, InstallationReasonMakeDependency) checkDependencies(pkgInfo.CheckDepends, InstallationReasonMakeDependency) } - if includeOptionalDepends { - checkDependencies(pkgInfo.OptionalDepends, InstallationReasonManual) - } } dfs(pkgInfo) diff --git a/src/bpmlib/general.go b/src/bpmlib/general.go index 9fa9ce1..9a0e1a1 100644 --- a/src/bpmlib/general.go +++ b/src/bpmlib/general.go @@ -12,7 +12,7 @@ import ( ) // InstallPackages installs the specified packages into the given root directory by fetching them from databases or directly from local bpm archives -func InstallPackages(rootDir string, forceInstallationReason InstallationReason, reinstallPackages bool, installRuntimeDependencies, installOptionalDependencies, forceInstallation, runChecks bool, verbose bool, packages ...string) (operation *BPMOperation, err error) { +func InstallPackages(rootDir string, forceInstallationReason InstallationReason, reinstallPackages bool, installRuntimeDependencies, forceInstallation, runChecks bool, verbose bool, packages ...string) (operation *BPMOperation, err error) { // Setup operation struct operation = &BPMOperation{ Actions: make([]OperationAction, 0), @@ -124,7 +124,7 @@ func InstallPackages(rootDir string, forceInstallationReason InstallationReason, } // Resolve dependencies - operation.ResolveDependencies(installRuntimeDependencies, installOptionalDependencies) + operation.ResolveDependencies(installRuntimeDependencies) if len(operation.UnresolvedDepends) != 0 { if !forceInstallation { return nil, DependencyNotFoundErr{operation.UnresolvedDepends} @@ -355,7 +355,7 @@ func CleanupCache(rootDir string, cleanupCompilationFiles, cleanupCompiledPackag } // UpdatePackages fetches the newest versions of all installed packages from -func UpdatePackages(rootDir string, syncDatabase bool, allowDowngrades bool, installOptionalDependencies, forceInstallation, runChecks, verbose bool) (operation *BPMOperation, err error) { +func UpdatePackages(rootDir string, syncDatabase, allowDowngrades, forceInstallation, runChecks, verbose bool) (operation *BPMOperation, err error) { // Sync databases if syncDatabase { err := SyncDatabase(verbose) @@ -420,7 +420,7 @@ func UpdatePackages(rootDir string, syncDatabase bool, allowDowngrades bool, ins } // Check for new dependencies in updated packages - operation.ResolveDependencies(true, installOptionalDependencies) + operation.ResolveDependencies(true) if len(operation.UnresolvedDepends) != 0 { if !forceInstallation { return nil, DependencyNotFoundErr{operation.UnresolvedDepends} diff --git a/src/bpmlib/operations.go b/src/bpmlib/operations.go index 359cae7..0d204c5 100644 --- a/src/bpmlib/operations.go +++ b/src/bpmlib/operations.go @@ -131,7 +131,7 @@ func (operation *BPMOperation) GetFinalActionSize(rootDir string) int64 { return ret } -func (operation *BPMOperation) ResolveDependencies(installRuntimeDepends, installOptionalDependencies bool) { +func (operation *BPMOperation) ResolveDependencies(installRuntimeDepends bool) { // Discover resolved virtual packages resolvedVirtualPackages := make(map[string]string) for _, value := range slices.Clone(operation.Actions) { @@ -167,7 +167,7 @@ func (operation *BPMOperation) ResolveDependencies(installRuntimeDepends, instal continue } - resolved, unresolved := ResolveDependencies(pkgInfo, resolvedVirtualPackages, installRuntimeDepends, installOptionalDependencies, operation.RootDir) + resolved, unresolved := ResolveDependencies(pkgInfo, resolvedVirtualPackages, installRuntimeDepends, operation.RootDir) // Append unresolved dependencies operation.UnresolvedDepends = append(operation.UnresolvedDepends, unresolved...)