From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 2002:a05:6512:3aa:0:0:0:0 with SMTP id v10csp394562lfp; Wed, 5 Feb 2020 09:51:44 -0800 (PST) X-Google-Smtp-Source: APXvYqxDO/VlTBP/RXbmnx1QRZQIqZNnU8FI4UByXBd4jZp+QfFT73OA7W8GINAPpaPfcB30bIQg X-Received: by 2002:a05:6214:80c:: with SMTP id df12mr32158924qvb.113.1580925104015; Wed, 05 Feb 2020 09:51:44 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1580925104; cv=none; d=google.com; s=arc-20160816; b=MWFIPJDovz+KwgtFu2fWsahIVQr35579ZYsnaByqo9EHnMi+FaZPqgu4Na4UF6PehT nZE/Ccci5rnI/IF+mxXpgaMCLIQyeMsmyXldTCw6wY9YS4msSQhCvyHKEEUOX6vAXu4g CVtbIPUX+Vkkeh6JDDfX1hTlNyImKCHzdptIbQQ3CR2aFwJrhN3xTWcKfZTSqO8HIkOi Q0yjUZRjdzH0DqCvQiWdSNZ/B7w08xvh+t5aoKOmomBYcxPOIQpVEcXT20lIcGA7xCsR wCnZc9crOMt47Q7+W9X/lkjDy0ltja4V1T9o0MHbOfluLR1/47ye/u85IRtKlh9GxPoH wicw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:reply-to:list-subscribe:list-help:list-post :list-archive:list-unsubscribe:list-id:precedence :content-disposition:content-transfer-encoding:user-agent :in-reply-to:mime-version:references:message-id:subject:to:from:date :dkim-signature; bh=uV5M0jxHDYGOUwiO7OTvaDi63qKi8FHxlrcbz5/XXZ8=; b=sBpLG9/m1cTU1qiY44VCxErsOh4EGufZ9pH/lFxBEKmYOMzQNZqSIeNHulhdVjsDVO 3zXNI/b/WlGNgLgk/RrJtylTSl6mztLjAMlR2t6xlmoUQEPvaWHqLKXxP9u+Uod1uhyh lokY0E3Q5fcKox4BDA/4A86zFgtOA57C7fSeYr8DPWcV9EMyEZyq9k3/tvQRmdIVl5vX VQaqaHiyhv+tnemkq+Jk5hqgqB5O4QT9d+Pp/+cknroQEb0RN2rkgl9MLbIc6/dJ8zLs xsPV1T4y1/oWjJ/7e6MQnSij8B0QiDC/L7s5kziZI2yhSYmrkqeCXEEDFezu7/G+yTmI BVog== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@redhat.com header.s=mimecast20190719 header.b=QhtI9F8y; spf=pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id m15si277831qkg.90.2020.02.05.09.51.43 for (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 05 Feb 2020 09:51:44 -0800 (PST) Received-SPF: pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=fail header.i=@redhat.com header.s=mimecast20190719 header.b=QhtI9F8y; spf=pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from localhost ([::1]:54566 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1izOpz-0007Da-Fs for alex.bennee@linaro.org; Wed, 05 Feb 2020 12:51:43 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:57229) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1izOpq-0007DD-GS for qemu-arm@nongnu.org; Wed, 05 Feb 2020 12:51:35 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1izOpo-00073k-DS for qemu-arm@nongnu.org; Wed, 05 Feb 2020 12:51:33 -0500 Received: from us-smtp-2.mimecast.com ([207.211.31.81]:21027 helo=us-smtp-delivery-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1izOpo-0006ze-9S for qemu-arm@nongnu.org; Wed, 05 Feb 2020 12:51:32 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1580925091; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=uV5M0jxHDYGOUwiO7OTvaDi63qKi8FHxlrcbz5/XXZ8=; b=QhtI9F8yB7KzidRPQu9x4v9e57kjbUp8DaaPQdWFO6P0uNBdtQeo33LLww576O3qLLHddt ikQmGKOUzn/T10ERC6TC0Msa8Y6P1rf56/U9tjJcxREhfFFrzvJ8eiW57thZYEr9HeY8gM F81/mI+p60+SrzeiGpoMwc1m1qNcQ5Y= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-35-DT4OWLtVNqCIfC3u79XmFw-1; Wed, 05 Feb 2020 12:51:29 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id E496214E4; Wed, 5 Feb 2020 17:51:27 +0000 (UTC) Received: from redhat.com (unknown [10.42.16.105]) by smtp.corp.redhat.com (Postfix) with ESMTPS id CD4A6790D7; Wed, 5 Feb 2020 17:51:17 +0000 (UTC) Date: Wed, 5 Feb 2020 17:51:15 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= To: Aleksandar Markovic Subject: Re: [PATCH for-5.0 3/4] Remove the core bluetooth code Message-ID: <20200205175115.GT2221087@redhat.com> References: <20191120091014.16883-1-thuth@redhat.com> <20191120091014.16883-4-thuth@redhat.com> <7a775153-f18d-e766-1d8f-33607f5fa05b@amsat.org> <2396dfd4-6c85-d17a-b358-bd058c757f14@redhat.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.13.3 (2020-01-12) X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-MC-Unique: DT4OWLtVNqCIfC3u79XmFw-1 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 207.211.31.81 X-BeenThere: qemu-arm@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Cc: Peter Maydell , Thomas Huth , libvir-list@redhat.com, Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , "qemu-devel@nongnu.org Developers" , qemu-arm , Gerd Hoffmann , Paolo Bonzini , Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= Errors-To: qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org Sender: "Qemu-arm" X-TUID: ZZ4QSTZUbYOc On Wed, Feb 05, 2020 at 06:40:31PM +0100, Aleksandar Markovic wrote: > On Sat, Feb 1, 2020 at 7:53 PM Philippe Mathieu-Daud=C3=A9 wrote: > > > > > >> @@ -1151,10 +1150,6 @@ for opt do > > > >> ;; > > > >> --enable-brlapi) brlapi=3D"yes" > > > >> ;; > > > >> - --disable-bluez) bluez=3D"no" > > > >> - ;; > > > >> - --enable-bluez) bluez=3D"yes" > > > >> - ;; > > > > > > > > Now than I'm bisecting over this commit, I realize removing this > > > > option was not a good idea, we should have done like commit > > > > cb6414dfec8 or 315d3184525: > > > > > > > > @@ -886,10 +885,6 @@ for opt do > > > > - --disable-uuid) uuid=3D"no" > > > > - ;; > > > > - --enable-uuid) uuid=3D"yes" > > > > - ;; > > > > ... > > > > + --enable-uuid|--disable-uuid) > > > > + echo "$0: $opt is obsolete, UUID support is always built" = >&2 > > > > + ;; > > > > > > Looks trivial ... so if it bugs you, just send a patch? > > > > I thought about it but this won't fix much, it is too late now. > > > > I simply wanted to share this bugged me so we try to avoid doing the > > same mistake again. > > >=20 > I vote for addition of a change similar to what Philippe described. >=20 > Furthermore, it looks to me the correct way would be to now do full > deprecation of --enable-bluez and --disable-bluez. This means adding > this to documentation (not related to bluetooth devices support), not > only a change in "configure". This also means that only after two next > full cycles these options could be removed. >=20 > True, this could have been done together with bluetooth devices > support deprecation (and in that case we could have deleted these > options right away), but it wasn't. Users don't have a crystal ball to > know that we assumed that --enable-bluez and --disable-bluez were part > of bluetooth devices support, and could rightfully complain about an > abrupt elimination of a compile time option. The deprecation policy is primarily intended for notifying of changes to QEMU's stable interfaces ( CLI, HMP, QMP) which affect behaviour and usage of QEMU at runtime & are liable to break apps managing QEMU. Changes to build time options have no strong reason to be subjected to the deprecation process. If we remove an option at build time the effect is noticed immediately and the solution is straightforward (stop passing the option). We have added / removed configure options at will with little negative feedback historically. We'll have far biggest changes coming to the build system in future too, with the introduction of meson. I don't think we want to constrain & complicate our work in modernizing the build system by declaring that any changes to it must go through deprecation. Regards, Daniel --=20 |: https://berrange.com -o- https://www.flickr.com/photos/dberrange= :| |: https://libvirt.org -o- https://fstop138.berrange.com= :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange= :|