From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (gnu.wildebeest.org [45.83.234.184]) by sourceware.org (Postfix) with ESMTPS id 0C9C43858CDB for ; Tue, 1 Nov 2022 22:20:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0C9C43858CDB Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=klomp.org Received: from reform (deer0x15.wildebeest.org [172.31.17.151]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id F278A3061FA4; Tue, 1 Nov 2022 23:20:33 +0100 (CET) Received: by reform (Postfix, from userid 1000) id 6A0202E8216E; Tue, 1 Nov 2022 23:20:33 +0100 (CET) Date: Tue, 1 Nov 2022 23:20:33 +0100 From: Mark Wielaard To: "Frank Ch. Eigler" Cc: elfutils-devel@sourceware.org Subject: Re: PATCH: Bug debuginfod/29472 followup Message-ID: References: <20221101142306.GL16441@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221101142306.GL16441@redhat.com> X-Spam-Status: No, score=-3033.0 required=5.0 tests=BAYES_00,JMQ_SPF_NEUTRAL,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi Frank, On Tue, Nov 01, 2022 at 10:23:06AM -0400, Frank Ch. Eigler via Elfutils-devel wrote: > On the users/fche/try-pr29472 branch, I pushed a followup to Ryan's > PR29472 draft from a bunch of weeks ago. It's missing some > ChangeLog's but appears otherwise complete. It's structured as Ryan's > original patch plus my followup that changes things around, so as to > preserve both contributions in the history. I paste the overall diff > here. > > There will be some minor merge conflicts between this and amerey's > section-extraction extensions that are also aiming for this release. > I'll be glad to deconflict whichever way. The section extraction extension was just pushed. But the testcase fails on some systems, which needs investigation. This is a fairly big patch which introduces even more new functionality, are you sure you want to target the 0.188 release of tomorrow? Are you sure the interface is correct? Is the sqlite "glob" pattern standardized? Can it be provided if the underlying server database isn't sqlite? I haven't read the whole diff yet. There are several refactorings which would be nice to see a separate patch. Why does debuginfod-client.c use json-c? Can't the server sent the json object as a normal char string? Why does the string from the server need to be interpreted as a json object and then turned into a string again? Cheers, Mark