Skip to content
Snippets Groups Projects

Fix "pmbootstrap config args /new/path"

Merged Imported Administrator requested to merge fix-config-aports into master

Fix writing config options that are stored as CSV-list. Without this patch:

$ pmbootstrap config aports /tmp/pmaports
[23:43:58] Config changed: aports='/tmp/pmaports'
$ pmbootstrap config aports
[PosixPath('/'), PosixPath('t'), PosixPath('m'), PosixPath('p'), PosixPath('/'), PosixPath('p'), PosixPath('m'), PosixPath('a'), PosixPath('p'), PosixPath('o'), PosixPath('r'), PosixPath('t'), PosixPath('s')]

Storing strings as CSV list isn't great and we probably want to implement this more elegantly, see the related issue. But let's fix this first.

Related: #2412 (closed)

Merging this directly to work towards unblocking pmaports CI (#2411 (closed)).

Merge request reports

Checking pipeline status.

Approval is optional

Merged by AdministratorAdministrator 8 months ago (Jul 17, 2024 10:02pm UTC)

Pipeline #188321 passed

Pipeline passed for b14a872f on master

Deployed to dep‎loy‎ 5 months ago

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading