From Oliver Smith to ~postmarketos/pmbootstrap-devel
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
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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"!) > >
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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. >
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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>
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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!
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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>
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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"
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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.
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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.
From Oliver Smith to ~postmarketos/pmbootstrap-devel
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