Re: Self dependent package, build profiles & buildd servers

2022-09-17 Thread Wookey
On 2022-09-17 10:25 +0800, Paul Wise wrote:
> On Fri, 2022-09-16 at 19:53 +0200, Fab Stz wrote:
> 
> > How does is this actually managed on the official buildd servers? How
> > does it actually know which DEB_BUILD_PROFILES to apply on each run?
> 
> The Debian buildds currently do not have support for build profiles, 
> for now build profiles are only for bootstrappers/porters/maintainers.
> 
> It sounds like you are going to have to fix the upstream build system
> to use the first build products during the second build product build.

As Paul says, build profiles will not help here if you have to do this
_every time_ you build the package, as opposed to doing it once, then
using that to build the real package the first time in order to get it
ready for the archive. i.e. build profiles are for bootstrapping, and the
results do not end up in the archive.

The normal way to fix this in Debian is for the debian build to run
the build twice, using the components built on the first run for the
second run. Can you do this easily, or do they have to be installed in
system paths to work?

You can also have the debian build run the build twice, one producing
pkg-A and one producing pkg-B, so that both end up in the
archive. Binutils is an example of a package that does this (building
both native and cross versions).


Wookey
-- 
Principal hats:  Debian, Wookware, ARM
http://wookware.org/


signature.asc
Description: PGP signature


Bug#1018148: RFS: neo-cli/0.6.1-1 [ITP] -- digital rain simulator

2022-09-17 Thread Mathias Gibbens
Hi Matthias,

On Tue, 2022-09-13 at 06:12 +0200, matthias.geiger1...@tutanota.de
wrote:
 Hi Mathias,

thanks for reaching out and taking the time for a review. I fixed
> all the improvements you mentioned and uploaded the new (old)
> version to mentors. 

  That's looking better. However, the Vcs- fields in d/control are
still unreachable. When I try to visit
"https://salsa.debian.org/werdahias/neo";, I get a not found error. It
looks like the correct URL is
"https://salsa.debian.org/werdahias/neo-cli";. You should update those
two fields so they are correct.

  After that's fixed, I'd say the package would be in good shape for
an upload. There's still some linitian messages that could be
addressed, but they aren't highlighting anything too terrible. (You
could probably fix them pretty easily and submit a patch/pull request
to the upstream developer.)

> Regarding cmatrix:
neo supports half-japananese characters/unicode, "appearing" text
like in The Matrix and 32 bit color support in addition.


  Thanks for that explanation. That's a good example of additional
information you could put in the package's extended description as
potential users would find it useful.

  Also, it's a good idea to always CC the bug address in your replies,
especially when they are ITP/RFS bugs. Doing so allows others who might
be watching the bug to stay up to date with any work that's being done
on it.

Mathias


signature.asc
Description: This is a digitally signed message part


Bug#982145: RFS: fuzzel/1.5.1-1 [ITP] -- Application launcher for wlroots based Wayland compositors

2022-09-17 Thread Peter Colberg
Dear Sway team,

Would anyone be willing to sponsor the initial upload of fuzzel? I am
a Debian Maintainer and would maintain the package myself thereafter.

On Sun, Oct 31, 2021 at 09:35:06PM +0100, Bastian Germann wrote:
> On Sat, 6 Feb 2021 15:10:26 -0500 Peter Colberg wrote:
> > It builds these binary packages:
> > 
> >   fuzzel - Application launcher for wlroots based Wayland compositors
> > 
> > Fuzzel fills a gap in the Debian swaywm ecosystem as a Wayland
> > equivalent for rofi in terms of ease of use, robustness, and simplicity.

> You should contact the team to sponsor this if you have not done this already.
> 
> The d/watch file does not work. The web interface might have changed.
> You can also use uscan's git mode.

The download likely failed due to pagination of older releases.
I have updated debian/watch to query the API instead [1].

> When you have fixed the d/watch file untag moreinfo.

Thanks for your review! I have pushed version 1.7.0-1 [2].

[1] 
https://salsa.debian.org/swaywm-team/fuzzel/-/commit/2cfcd218545134ba7b1c5ab479b76f29a48e46e6
[2] 
https://salsa.debian.org/swaywm-team/fuzzel/-/commit/e3f6004d4c6a1de1e6c1b599d00b5c7fb4f35408

Thanks,
Peter



Bug#1018148: RFS: neo-cli/0.6.1-1 [ITP] -- digital rain simulator

2022-09-17 Thread matthias . geiger1024
 I fixed the VCS fields in d/control and amended the extended description. The 
lintinan info about the manpage was already reported upstream by me. I think it 
should be good now. Thanks for taking the time to review/sponsor it.
---
Matthias Geiger (werdahias)