Skip to content

args: remove -m and -mp

Administrator requested to merge remove-m-mp-args into master

Remove arguments to set postmarketOS and Alpine mirrors to reduce code complexity and to make the help output a bit smaller. Most users don't need to use this, and if they do, it makes much more sense to configure the mirrors once in the config instead of doing it with every pmbootstrap run.

Split from !2361 (merged) so we can discuss it here some more. @minlexx was actually using this, not sure if he is happy with what I proposed to do instead. Below is again my proposal.

I would think that users don't need to switch mirrors often and rather configure once which mirrors they want to use.

I would suggest to create separate scripts that you run once when you want to switch the mirror. The benefit is then that you can just run "pmbootstrap" directly without going through a wrapper script.

$ cat set_mirror_to_local.sh
#!/bin/sh -ex
pmb="python3 ~/projects/python/pmbootstrap/pmbootstrap.py"
$pmb config mirrors.pmaports "http://192.168.0.200/postmarketos/"
$pmb config mirrors.alpine "http://192.168.0.200/alpine/"
$ cat set_mirror_reset.sh
#!/bin/sh -ex
pmb="python3 ~/projects/python/pmbootstrap/pmbootstrap.py"
$pmb config -r mirrors.pmaports
$pmb config mirrors.alpine "http://mirror.yandex.ru/mirrors/alpine/"

If you really want to go with a wrapper, it would look like this:

$ cat your_fancy_pmb_wrapper.sh
#!/bin/sh -e
pmb="python3 ~/projects/python/pmbootstrap/pmbootstrap.py"
# $pmb -q config -r mirrors.pmaports
$pmb -q config mirrors.pmaports "http://192.168.0.200/postmarketos/"
# $pmb -q config mirrors.alpine "http://mirror.yandex.ru/mirrors/alpine/"
$pmb -q config mirrors.alpine "http://192.168.0.200/alpine/"
$pmb "$@"

Merge request reports