Hi hexaheximal,

(please add the list in CC next time)

hexaheximal:
> Thanks for reviewing the patch!
> 
>> Sorry but I don't see how this fixes things. Can you add a test case
>> that shows it?
> 
> The machine I'm running pmbootstrap on is STRICTLY behind a proxy (non-proxied connections won't work), pmbootstrap didn't work before, and with this patch it does.

I believe you, and I'm happy that pmbootstrap works behind a proxy with
this. There's value in having a test case though, it would make more

On Mon Aug 14, 2023 at 9:36 AM CEST, Oliver Smith wrote:
> On Sun Aug 13, 2023 at 1:41 AM CEST, Caleb Connolly wrote:
> > Add a new "auto_checksum" config option which, when set, will make
> > pmbootstrap fix checksums for local sources in-place before building a
> > package.
> >
> > This _drastically_ speeds up development when working with files that
> > are local to pmaports.
> >
> > It isn't done automatically for remote sources as there is still some
> > value in having these error out (e.g. catching corrupt downloads).
> > Additionally, those sources shouldn't change regularly (if you're doing
> > local development like this, please use "pmbootstrap build --src"!)
> >

On Tue Aug 8, 2023 at 8:57 PM CEST, Luca Weiss wrote:
> On Sonntag, 6. August 2023 20:43:31 CEST Oliver Smith wrote:
> > Cache the compiler output of rust code with sccache, like we use ccache
> > for c code.
> > 
> > I've considered using sccache to completely replace ccache since it can
> > cache output of C/C++ code too. But let's not do it for now since ccache
> > doesn't need to run a daemon in the background that needs to be stopped
> > when shutting down / zapping. Also it would need more refactoring.
>
> Sounds like a decent idea. Any idea about the performance difference vs ccache 
> for C/C++ code? Killing the daemon is already implemented so that's not an 
> obstacle anymore.
>

On Sat, 12 Aug 2023 15:55:26 +0000, Raymond Hackley wrote:
> 


Applied, thanks!

[1/1] README.md: add armv7 to supported architecture
      commit: 9045bfb2b912ddbc1c53a70abb4ae4d540deb1ef

Best regards,
-- 
Oliver Smith <ollieparanoid@postmarketos.org>

On Sun, 06 Aug 2023 20:43:27 +0200, Oliver Smith wrote:
> Refactor /mnt/pmbootstrap-* to have all these directories as
> subdirectories of /mnt/pmbootstrap. Fix that these dirs did not get
> removed when installation was done.
> 
> Add one more dir in /mnt/pmbootstrap/ for using sccache with rust,
> similar to how we already use ccache for C/C++ code. It speeds up
> subsequent builds greatly for native builds, and for native scripts
> built during cross builds.
> 
> [...]

Applied, thanks!

On Sun, 13 Aug 2023 22:20:45 +0200, lauren@selfisekai.rocks wrote:
> this seems to be parsed correctly, other than this one check
> 
> 

Applied, thanks!

[1/1] parse: allow "-r" in pkgver
      commit: 753e563ccc57a31758905b239b95008e3e19466a

Best regards,
-- 
Oliver Smith <ollieparanoid@postmarketos.org>

On Sun Aug 13, 2023 at 10:20 PM CEST,  wrote:
> From: "Lauren N. Liberda" <lauren@selfisekai.rocks>
>
> this seems to be parsed correctly, other than this one check
>
> bug: https://gitlab.com/postmarketOS/pmbootstrap/-/issues/2260

Thanks!

Apparently the reason for adding it was this, from 5 years ago in
https://gitlab.com/postmarketOS/pmbootstrap/-/issues/839:

"when the version currently contains -r, it will confuse pmbootstrap. @ata2001 reported:
 it did not crash, but rebuilt the package every time, then installed the old version"

On Sun Aug 13, 2023 at 1:41 AM CEST, Caleb Connolly wrote:
> Add a new "auto_checksum" config option which, when set, will make
> pmbootstrap fix checksums for local sources in-place before building a
> package.
>
> This _drastically_ speeds up development when working with files that
> are local to pmaports.
>
> It isn't done automatically for remote sources as there is still some
> value in having these error out (e.g. catching corrupt downloads).
> Additionally, those sources shouldn't change regularly (if you're doing
> local development like this, please use "pmbootstrap build --src"!)
>
> Enable this option by default, it doesn't introduce any additional risk.

On Sun Aug 13, 2023 at 1:41 AM CEST, Caleb Connolly wrote:
> Add a new "auto_checksum" config option which, when set, will make
> pmbootstrap fix checksums for local sources in-place before building a
> package.
>
> This _drastically_ speeds up development when working with files that
> are local to pmaports.
>
> It isn't done automatically for remote sources as there is still some
> value in having these error out (e.g. catching corrupt downloads).
> Additionally, those sources shouldn't change regularly (if you're doing
> local development like this, please use "pmbootstrap build --src"!)
>
> Enable this option by default, it doesn't introduce any additional risk.

On Sat Aug 12, 2023 at 5:55 PM CEST, Raymond Hackley wrote:
> ---
>  README.md | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Assuming you have verified that it works:

Reviewed-by: Oliver Smith <ollieparanoid@postmarketos.org>

Regarding riscv64: in theory it should work, but let's add it after
somebody has confirmed that it works.

> diff --git a/README.md b/README.md