Skip to content

Conversation

@ThatOneCalculator
Copy link
Contributor

As reported here: https://aur.archlinux.org/packages/upcloud-cli#comment-1032759

The current PKGBUILD results in

==> Starting package()...
/home/kainoa/.cache/yay/upcloud-cli/PKGBUILD: line 12: cd: null directory
==> ERROR: A failure occurred in package().
    Aborting...
 -> error making: upcloud-cli-exit status 4

Removing that cd line results in a successful installation.

As reported here: https://aur.archlinux.org/packages/upcloud-cli#comment-1032759

The current PKGBUILD results in 

```
==> Starting package()...
/home/kainoa/.cache/yay/upcloud-cli/PKGBUILD: line 12: cd: null directory
==> ERROR: A failure occurred in package().
    Aborting...
 -> error making: upcloud-cli-exit status 4
```

Removing that `cd` line results in a successful installation.
@ThatOneCalculator ThatOneCalculator requested a review from a team as a code owner August 11, 2025 17:20
Copy link
Contributor

@villevsv-upcloud villevsv-upcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello! Thanks for the report & contribution. I've added a suggestion of adding the $srcdir to source path. package() seems to work either way, but better to explicitly state the dir.

@villevsv-upcloud villevsv-upcloud self-requested a review August 12, 2025 11:36
@villevsv-upcloud villevsv-upcloud changed the title fix: PKGBUILD null directory fix(release): PKGBUILD null directory Aug 12, 2025
@villevsv-upcloud villevsv-upcloud merged commit 0eeac69 into UpCloudLtd:main Aug 12, 2025
2 checks passed
@ThatOneCalculator ThatOneCalculator deleted the patch-1 branch August 12, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants