Re: [docs] [PATCH] Fix typos and spacing

["Michael Opdenacker" <michael.opdenacker@bootlin.com>]

Hi Quentin,

Many thanks for your review!

On 3/29/21 5:19 PM, Quentin Schulz wrote:
>> @@ -2111,8 +2111,7 @@ accomplished using fakeroot.
>>     under fakeroot. Otherwise, the task cannot run root-only operations,
>>     and cannot see the fake file ownership and permissions set by the
>>     other task. You need to also add a dependency on
>> -   virtual/fakeroot-native:do_populate_sysroot
>> -   , giving the following:
>> +   virtual/fakeroot-native:do_populate_sysroot, giving the following:
>>     ::
> This is usually the tell of a bad outcome of the migration tool. I'm
> pretty sure what was meant here is:
> other task. You need to also add a dependency on
> ``virtual/fakeroot-native:do_populate_sysroot``, giving the following:

Oops, this definitely makes sense.

>>  
>>     It is not a requirement that a layer name begin with the prefix
>> -   meta-
>> -   , but it is a commonly accepted standard in the Yocto Project
>> +   meta-, but it is a commonly accepted standard in the Yocto Project
> Same here, ``meta-``.
>
> Basically, you just encountered the files we haven't manually reviewed
> yet :)
>
> Outside of obvious typos, anything with a weird indentation/syntax
> usually requires manual inspection. I used to compare with 3.0 docs to
> check what was intended and then fix it to match whatever was the state
> before.
>
> I would say it's better to leave those indentation/syntax issues as is
> and take the time to fix them properly because they are pretty easy to
> spot once your brain has picked up the patterns :)


Agreed. I'll drop these in the current patch and will address them
separately. Indeed, I've seen a few other looking like these ones.

Thanks again!

Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com