On Mon, Apr 25, 2022, at 11:44 AM, Seebs wrote: > On Mon, 25 Apr 2022 10:49:24 -0400 > "C. Andy Martin" <[email protected]> wrote: > >> - msg->result = RESULT_ABORT; >> - goto op_exit; >> + if (msg->nlink == 1) { >> + msg->result = >> RESULT_ABORT; >> + goto op_exit; >> + } > > I'm distrustful of this because I feel like, in the case where we have > apparently-stale information in the database, we really should be coming > back with "we don't have valid database information", not yielding > what's probably incorrect/old data. > > I'm also concerned a bit because nlink is never 1 for a directory... > > -s
I agree this is not the ideal solution. However, I was basing this compromise on this commit: commit 691a2304c8c29a3b7d5977908ef82aaf188f5ba0 Author: Seebs <[email protected]> Date: Fri Apr 13 14:37:34 2018 -0500 Less chatty debugging Stop producing the path mismatch messages for things with multiple links. We could probably make this smarter, but for now this is sufficient to reduce the spam. pseudo_debug(0, ...) now produces a message, as a special case. Signed-off-by: Seebs <[email protected]> The issue is w/out the change, my builds occasionally abort due to races -Andy
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#164839): https://lists.openembedded.org/g/openembedded-core/message/164839 Mute This Topic: https://lists.openembedded.org/mt/90686441/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
