From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-x231.google.com (mail-lj1-x231.google.com [IPv6:2a00:1450:4864:20::231]) by sourceware.org (Postfix) with ESMTPS id 97D0E3858C50 for ; Thu, 4 Apr 2024 08:12:45 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 97D0E3858C50 Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=suse.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 97D0E3858C50 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2a00:1450:4864:20::231 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1712218367; cv=none; b=hp7si1xFiEHvQ86/rtwQQsoGc5PPFkoa4au8vTSxCmRfQ9TWt0uImv16key/4F1ipF2NsVv31Z2ZyIpAvloNU3TV8Vhn0EVcUQJV0GnMEgmuZIEqiyXrBTUnx8ao4LLS+A/B+CY/ZElbtchnU8ZpQZ0uMlbPfPV1kWvQy1aFG98= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1712218367; c=relaxed/simple; bh=ruh8wSnzCeo2mbBQLBc0eImhcg2/e6vG67GpdqcWa8I=; h=DKIM-Signature:Message-ID:Date:MIME-Version:Subject:To:From; b=gX1oMnNl6i+DOM25vc65FvElgyzqEkIWmGwSNXLKwLjmkpvinwiMPGs9VxnYwq1PY1/a7mqh3Ird4j8N9MTANXHdI3Zn8z+DRvXFUYV+FAkKDqwnA/j6VPw2vq7vaRCECmjGExdf/P9I1Y6UFlvpVKRE6RSDbjbi/th+GhNF45Q= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-lj1-x231.google.com with SMTP id 38308e7fff4ca-2d718efedb2so10593711fa.0 for ; Thu, 04 Apr 2024 01:12:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1712218364; x=1712823164; darn=sourceware.org; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ruh8wSnzCeo2mbBQLBc0eImhcg2/e6vG67GpdqcWa8I=; b=PiWJjxq68neRSJRpqbb1DMVgbIZG1k4cThNfKA/vQRfMkHhdtnj4e50qbfB8YgsFe8 ogDJzwO99bHQbrO5MWQkO2M8EqoVGMJNolTKtfcXlNSMm3zY+AYNAtcf4ZB4hVYaAYqN wryF0xtN8xw0iI83xKR81zAN58H28K/J7El0CezS1+NI+hfBUriLajnuchwcfYNQrZJb PxxhGpuHCc81cdFdFm0Iwbc5Q/P5hzuEUcwij16fY/I0LA8CyeWOw9VXdQCjpI9zKug/ 1QeOSgA8+0uC+ZPd/8/gOFwFY5aTemCADdFK6snPV+txI/VE4b62Poocfkx6yeQXZHYN hdTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712218364; x=1712823164; h=content-transfer-encoding:in-reply-to:autocrypt:from:references:cc :to:content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ruh8wSnzCeo2mbBQLBc0eImhcg2/e6vG67GpdqcWa8I=; b=nyspwMO6DMQseYp+hxeCbwAbGNq5iHgVttNYtphTwWlitsLaWeq4GSeFlWMnN/0PPj 3aAhvnU8P4KV4e4lsqmzbVFhVkb0PI1RY57LBxOSGvgvK//F15VhFrVJAyGXwR7b01yT Av8gayVJpnDsrnXk7QJfp+uBMnzQe9KPbD+yL8SwYFGGGlrqWxhwaiDLVE2xNegdDRua xL06u8yU42cvgKcjfqmLOiLf63r/Ex109i4cOZAALp20yKaLoMsDM7V6PjYnOMTMeINe stEu7TyeOOOpHcY0V5Yy4jEtG1okch7lschumWhIUOj4zHDppFCFCr7+PVLnQz+PzWhS apYw== X-Forwarded-Encrypted: i=1; AJvYcCVDiL1XCc/28NBRQnDhx23fDT35tSD7LzqejU+e0drHqP5atADFww14sl7sZRa/ZAdlqkoPtvx0wlevA+ABxGCQgXw= X-Gm-Message-State: AOJu0Yx2J7sYYDM/bgWmmgcaccdYqBfeDJOAlGpm+PJTmEi2mePaNQYQ t4mSjmiEhkma3GFRaJgREk5PVm/vz6KhVNmUk54BMnTB8yiRHdlHlC/pnSdMIg== X-Google-Smtp-Source: AGHT+IFAcb+3wlCAnVHgx5jOaHd66Ue+6aIdGRduF3fs5S8Xv3XdqoJtflQiwe2p6pBsQFRGdfpfZw== X-Received: by 2002:ac2:5dd3:0:b0:513:cfcd:f25f with SMTP id x19-20020ac25dd3000000b00513cfcdf25fmr1444671lfq.54.1712218364010; Thu, 04 Apr 2024 01:12:44 -0700 (PDT) Received: from [10.156.60.236] (ip-037-024-206-209.um08.pools.vodafone-ip.de. [37.24.206.209]) by smtp.gmail.com with ESMTPSA id xe3-20020a170907318300b00a471481ef3csm8659545ejb.124.2024.04.04.01.12.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 04 Apr 2024 01:12:43 -0700 (PDT) Message-ID: <0d343d67-33e0-4f63-b385-8ae360bc68dc@suse.com> Date: Thu, 4 Apr 2024 10:12:43 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Patches submission policy change Content-Language: en-US To: Christophe Lyon Cc: Maxim Kuvyrkov , Thiago Bauermann , Adhemerval Zanella , binutils@sourceware.org, GCC Mailing List , gdb@sourceware.org, Nick Clifton , Richard Biener , Jakub Jelinek , Joel Brobecker , Carlos O'Donell References: From: Jan Beulich Autocrypt: addr=jbeulich@suse.com; keydata= xsDiBFk3nEQRBADAEaSw6zC/EJkiwGPXbWtPxl2xCdSoeepS07jW8UgcHNurfHvUzogEq5xk hu507c3BarVjyWCJOylMNR98Yd8VqD9UfmX0Hb8/BrA+Hl6/DB/eqGptrf4BSRwcZQM32aZK 7Pj2XbGWIUrZrd70x1eAP9QE3P79Y2oLrsCgbZJfEwCgvz9JjGmQqQkRiTVzlZVCJYcyGGsD /0tbFCzD2h20ahe8rC1gbb3K3qk+LpBtvjBu1RY9drYk0NymiGbJWZgab6t1jM7sk2vuf0Py O9Hf9XBmK0uE9IgMaiCpc32XV9oASz6UJebwkX+zF2jG5I1BfnO9g7KlotcA/v5ClMjgo6Gl MDY4HxoSRu3i1cqqSDtVlt+AOVBJBACrZcnHAUSuCXBPy0jOlBhxPqRWv6ND4c9PH1xjQ3NP nxJuMBS8rnNg22uyfAgmBKNLpLgAGVRMZGaGoJObGf72s6TeIqKJo/LtggAS9qAUiuKVnygo 3wjfkS9A3DRO+SpU7JqWdsveeIQyeyEJ/8PTowmSQLakF+3fote9ybzd880fSmFuIEJldWxp Y2ggPGpiZXVsaWNoQHN1c2UuY29tPsJgBBMRAgAgBQJZN5xEAhsDBgsJCAcDAgQVAggDBBYC AwECHgECF4AACgkQoDSui/t3IH4J+wCfQ5jHdEjCRHj23O/5ttg9r9OIruwAn3103WUITZee e7Sbg12UgcQ5lv7SzsFNBFk3nEQQCACCuTjCjFOUdi5Nm244F+78kLghRcin/awv+IrTcIWF hUpSs1Y91iQQ7KItirz5uwCPlwejSJDQJLIS+QtJHaXDXeV6NI0Uef1hP20+y8qydDiVkv6l IreXjTb7DvksRgJNvCkWtYnlS3mYvQ9NzS9PhyALWbXnH6sIJd2O9lKS1Mrfq+y0IXCP10eS FFGg+Av3IQeFatkJAyju0PPthyTqxSI4lZYuJVPknzgaeuJv/2NccrPvmeDg6Coe7ZIeQ8Yj t0ARxu2xytAkkLCel1Lz1WLmwLstV30g80nkgZf/wr+/BXJW/oIvRlonUkxv+IbBM3dX2OV8 AmRv1ySWPTP7AAMFB/9PQK/VtlNUJvg8GXj9ootzrteGfVZVVT4XBJkfwBcpC/XcPzldjv+3 HYudvpdNK3lLujXeA5fLOH+Z/G9WBc5pFVSMocI71I8bT8lIAzreg0WvkWg5V2WZsUMlnDL9 mpwIGFhlbM3gfDMs7MPMu8YQRFVdUvtSpaAs8OFfGQ0ia3LGZcjA6Ik2+xcqscEJzNH+qh8V m5jjp28yZgaqTaRbg3M/+MTbMpicpZuqF4rnB0AQD12/3BNWDR6bmh+EkYSMcEIpQmBM51qM EKYTQGybRCjpnKHGOxG0rfFY1085mBDZCH5Kx0cl0HVJuQKC+dV2ZY5AqjcKwAxpE75MLFkr wkkEGBECAAkFAlk3nEQCGwwACgkQoDSui/t3IH7nnwCfcJWUDUFKdCsBH/E5d+0ZnMQi+G0A nAuWpQkjM1ASeQwSHEeAWPgskBQL In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-3025.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_NONE,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 03.04.2024 15:11, Christophe Lyon wrote: > On Wed, 3 Apr 2024 at 10:30, Jan Beulich wrote: >> >> On 03.04.2024 10:22, Christophe Lyon wrote: >>> Dear release managers and developers, >>> >>> TL;DR: For the sake of improving precommit CI coverage and simplifying >>> workflows, I’d like to request a patch submission policy change, so >>> that we now include regenerated files. This was discussed during the >>> last GNU toolchain office hours meeting [1] (2024-03-28). >>> >>> Benefits or this change include: >>> - Increased compatibility with precommit CI >>> - No need to manually edit patches before submitting, thus the “git >>> send-email” workflow is simplified >>> - Patch reviewers can be confident that the committed patch will be >>> exactly what they approved >>> - Precommit CI can test exactly what has been submitted >>> >>> Any concerns/objections? >> >> Yes: Patch size. And no, not sending patches inline is bad practice. > Not sure what you mean? Do you mean sending patches as attachments is > bad practice? Yes. It makes it difficult to reply to them (with proper reply context). >> Even assuming sending patches bi-modal (inline and as attachment) works >> (please indicate whether that's the case), it would mean extra work on >> the sending side. >> > For the CI perspective, we use what patchwork is able to detect as patches. > Looking at recent patches submissions, it seems patchwork is able to > cope with the output of git format-patch/git send-email, as well as > attachments. > There are cases where patchwork is not able to detect the patch, but I > don't know patchwork's exact specifications. Question was though: If a patch was sent inline plus attachment, what would CI use as the patch to apply? IOW would it be an option to attach the un-stripped patch, while inlining the stripped one? Jan