ACK/CMNT: [SRU][L/M][PATCH] UBUNTU: [Packaging] introduce a separate linux-lib-rust package

Andrea Righi andrea.righi at canonical.com
Tue May 9 11:16:38 UTC 2023


On Tue, May 09, 2023 at 01:00:52PM +0200, Luke Nowakowski-Krijger wrote:
> Acked-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger at canonical.com>
> 
> Looks good to me, I pointed out below a line that potentially adds some
> whitespace to maybe be aware of when applying.
> 
> > @@ -386,7 +385,7 @@ endif
> >         #
> >         # Remove files which are generated at installation by postinst,
> >         # except for modules.order and modules.builtin
> > -       #
> >
> 
> I think there is some whitespace being added here? Its perhaps not worth a
> resend but if anyone remembers when applying..

I actually removed some whitespaces here. Sorry, my vim is screaming
when there are trailing whitespaces, so I noticed them and removed as a
bonus cleanup, I should have mentioned it... :)

Thanks,
-Andrea

PS from my ~/.vimrc:

 let c_space_errors=1
 highlight WhitespaceEOL ctermbg=red guibg=red
 match WhitespaceEOL /\s\+$/



More information about the kernel-team mailing list