<conchuod>
yo palmer you about? gonna send pr in a minute or two so lmk if i do it the way you want.
<conchuod>
anything specific you want /other than/ the signed tag & what request-pull outputs?
<palmer>
I'm here
<palmer>
That should be fine
prabhakarlad has quit [Quit: Client closed]
<conchuod>
I'll be around for an hour or two so if something's obviously wrong lmk & I'll resend.
<palmer>
OK
gdd has joined #riscv
elastic_dog has quit [Ping timeout: 258 seconds]
elastic_dog has joined #riscv
<palmer>
LGTM, I'll go figure out how to merge it ;)
<palmer>
conchuod: I'm getting "fatal: Commit 5e757deddd91 does not have a GPG signature.", but I see a signature on the tag so I think's OK on your end. I'm going to go mess around with GPG a bit, I can never quite figure it out
<conchuod>
hmm, idk do you need to import my key? but the commit itself isn't signed - just the tag
<palmer>
IDK
<palmer>
I can turn off merge.verifySignatures, but then even without your key it'll let me merge it
<conchuod>
git tag -v dt-fixes-for-palmer-5.19-rc3 should verify it
<conchuod>
But ye, idk what the proper flow is there? Should I just sign the commits that I apply too?
<conchuod>
Do you sign commits by default?
<palmer>
I do sign commits, but I don't think that's common
<conchuod>
hmm all the maintainers guide seems to say about verification is "Maintainers at all levels should be using signed tags on their pull requests, and upstream maintainers should verify the tags when pulling branches"
littlebobeep has joined #riscv
<palmer>
ya, I think it's common to just sign tags
<palmer>
but not sure how to verify them ;)
<conchuod>
git tag -v dt-fixes-for-palmer-5.19-rc3
<palmer>
ya, but I don't have it locally
<palmer>
and IIUC that's the right way to pull stuff
<palmer>
(though I guess my repos are polluted with crazy tags anyway)
<muurkha>
you can use git fetch to fetch it without merging it into your current branch?
<conchuod>
You mean, the tag isn't local or the key isn't?
<palmer>
the tag isn't local
<palmer>
so I just did "git pull git://.../conor/linux.git dt-fixes-for-palmer"
<conchuod>
idk palmer, I guess you could do "b4 pr"? At least that's what I did to check that it would work & that checks the signature
<palmer>
OK, probably a good excuse to start using b4 ;)
<palmer>
it's as much confusion on my end as anything else
<palmer>
anything new like this will probably be a bit rocky for a bit, but we'll figure it out
<conchuod>
Ye I am ~always around so if you need anything from me, come knocking.
ashln has quit [Ping timeout: 246 seconds]
<conchuod>
~ in the maths not the c sense
ashln has joined #riscv
<palmer>
cool, thanks. I think for now just picking up the microsemi stuff is a good start, that'll help some and it'll help clean the pipes so other folks can start doing the same for their targets
<conchuod>
Once we get rid of all the dt warnings, I'd like to try & help keep it that way.
littlebobeep has joined #riscv
<palmer>
well, once they're gone they'll stay gone because I won't merge anything that adds them (and Rob can add it to his checker, too)
<palmer>
but having them be almost-gone means I can't fail the merges on that ;)
<conchuod>
For whats /currently/ enabled in defconfig it's just Atuls stuff
<conchuod>
But then there's the k210, which I'll prob have a punt at this weekend - but that's not enabled by defconfig (and neither is the starfive but that had no errors)
pecastro has quit [Ping timeout: 256 seconds]
littlebobeep has quit [Remote host closed the connection]