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