Skip to content

Draft: install_pmbootstrap: add_remote_origin_original: reorder commands

Administrator requested to merge wip into master

Note: This will probably not be needed if the move to SourceHut is successful.

This script was updated to work with a new security feature of Git, where it checks ownership (in Unix permissions) of the repository. Since the script only runs in CI, the pmaports directory can be marked as safe.

Originally, the commands to do this were placed in the middle of the function, before adding the remote, but after checking for an existing remote. This is problematic because the security feature would block the check, causing the remote to be added even when it already exists.

These commands must go at the beginning of the function, so that the security feature doesn't block any Git command.

Why is this still a draft?

Some weird chowning changes the git repository to be owned by 1000:1000 (pmos:pmos) instead of 0:0 (root:root). This persists in cache and causes the error shown here.

I made a partial fix here but when one job fails, every job after it fails as well.

Edited by Administrator

Merge request reports

Loading