From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: =?UTF-8?Q?Bj=C3=B6rn?= Bidar via "Bug reports for GNU Emacs, the Swiss army knife of text editors" Newsgroups: gmane.emacs.bugs Subject: bug#74413: [PATCH] Allow to store and read repository information of VCS builds Date: Mon, 18 Nov 2024 21:31:43 +0200 Message-ID: <49283.2275618397$1731958353@news.gmane.org> References: <86frnovhg8.fsf@gnu.org> <867c90vbfk.fsf@gnu.org> <861pz8uzyo.fsf@gnu.org> Reply-To: =?UTF-8?Q?Bj=C3=B6rn?= Bidar Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="9654"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: luangruo@yahoo.com, 74413@debbugs.gnu.org, stefankangas@gmail.com To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Mon Nov 18 20:32:26 2024 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1tD7U1-0002Mq-TY for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 18 Nov 2024 20:32:26 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tD7Tk-0006Aj-NS; Mon, 18 Nov 2024 14:32:10 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tD7Te-0006AT-RC for bug-gnu-emacs@gnu.org; Mon, 18 Nov 2024 14:32:03 -0500 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1tD7Te-0002E6-Fg for bug-gnu-emacs@gnu.org; Mon, 18 Nov 2024 14:32:02 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=debbugs.gnu.org; s=debbugs-gnu-org; h=MIME-Version:Date:References:In-Reply-To:From:To:Subject; bh=ArpbF+DDVtKdLS/chJfp8QDQhA00PvAFtqhUG074XGU=; b=sjd0DrgK/CmRh+bw24iovjfwuKWGsBAEwZDMSkXI5u5F4V8YZwbnp4609rvkcf0STyrMbLPXrFNHVByFTnbBvdrDZ/q3XrXOU8Ch249FaXPd5HSHbRzXj63Pe/kzFmpI/1BjgprmvfTSlPu9vRMnLdqUkYi38ckJLsoIwqAm2KpTCqQjd0FolKQhTIxu47pww5qip25JaNdOsfYT2AkEL8hDGVDeEUTXi0wpwU7m9GrVmx9khgA9z/IVhXeKAIu9M+xW42q/+WMji6lr6qxWtNVG1hk/Nkg+sBaZkJv7LqgVRYWIr1STb55rW+JnaPIdvk3smJzOjuBO9lChJ7D0Sg==; Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1tD7Td-0002qN-W8 for bug-gnu-emacs@gnu.org; Mon, 18 Nov 2024 14:32:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: =?UTF-8?Q?Bj=C3=B6rn?= Bidar Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 18 Nov 2024 19:32:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 74413 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 74413-submit@debbugs.gnu.org id=B74413.173195830910909 (code B ref 74413); Mon, 18 Nov 2024 19:32:01 +0000 Original-Received: (at 74413) by debbugs.gnu.org; 18 Nov 2024 19:31:49 +0000 Original-Received: from localhost ([127.0.0.1]:35217 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1tD7TR-0002pt-EU for submit@debbugs.gnu.org; Mon, 18 Nov 2024 14:31:49 -0500 Original-Received: from thaodan.de ([185.216.177.71]:37142) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1tD7TO-0002pi-SU for 74413@debbugs.gnu.org; Mon, 18 Nov 2024 14:31:48 -0500 Original-Received: from odin (dsl-trebng12-50dc7b-49.dhcp.inet.fi [80.220.123.49]) by thaodan.de (Postfix) with ESMTPSA id 41035D00071; Mon, 18 Nov 2024 21:31:45 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=thaodan.de; s=mail; t=1731958305; bh=T2Mp3lxzCG89MgnvduMwdUgLHQ9BVogYFxGPZZkZSLA=; h=From:To:Cc:Subject:In-Reply-To:References:Date; b=ePQ9n8NRfULMREpvM+UdLUnSBgt+O0etPZ6KeysWosGDOohWqunbH4eeKtv3Dcu8t /d/OUn3+PRYtMFUEY/seA742s2JK53oYvZGuEmDoEEFruDiX+7rUK/q8gaLI3/7Kw5 faOoH7sz5XZ2DJcTdwAAO9pmh5C3lYOMQ9GFgn1Nxg67vb9/gc5X20ITxfBU5qRjl1 amhzIJuuC6hRI80+AQfwt2sP1Hq8ow/P2r3RrInzKhUtWOh1biJiKiS6bquERh/4q8 mZ0oLzPPSk/cyF0SIoa2ztQtSV9oeT4jZZ3V/TLV1W3f6qi2yfwcmzM65NQLMQ10Bg ywaWZofGjwRs/UA6x6VbWu3AnEZrmATsrS7u2wWEUlipqsKMjprZM/7UeI+vVyxu+I 1xW9IU+M/otH37GsYiNf9S6ZV8dPvZI+oNXpbkJ6+oYosb55K7lmv/lrMHidV/pyaU pmqtxV8P5nbzddwH+EwfP1RfGBxnSGFRJVZZC6GRgY3pCI1imv+69vuT4UV9EXBW1e 3k/bWhaZEg82+TYByktP9V0MOR7RwkCYjB5EYt9u44kJsVyIcqUi9bGN6WcDITD9Jr r1wjGjChIZmHxID85L6qX36M/j/IOqMv+vBdwl2HZrjyde76IsWowO+ckpb+NucTJH rWmZ9kcL7ftkEIwF0L2X0w/w= In-Reply-To: <861pz8uzyo.fsf@gnu.org> (Eli Zaretskii's message of "Mon, 18 Nov 2024 21:03:11 +0200") Autocrypt: addr=bjorn.bidar@thaodan.de; prefer-encrypt=nopreference; keydata= mDMEZNfpPhYJKwYBBAHaRw8BAQdACBEmr+0xwIIHZfIDlZmm7sa+lHHSb0g9FZrN6qE6ru60JUJq w7ZybiBCaWRhciA8Ympvcm4uYmlkYXJAdGhhb2Rhbi5kZT6IlgQTFgoAPgIbAwULCQgHAgIiAgYV CgkICwIEFgIDAQIeBwIXgBYhBFHxdut1RzAepymoq1wbdKFlHF9oBQJk1/YmAhkBAAoJEFwbdKFl HF9oB9cBAJoIIGQKXm4cpap+Flxc/EGnYl0123lcEyzuduqvlDT0AQC3OlFKm/OiqJ8IMTrzJRZ8 phFssTkSrrFXnM2jm5PYDoiTBBMWCgA7FiEEUfF263VHMB6nKairXBt0oWUcX2gFAmTX6T4CGwMF CwkIBwICIgIGFQoJCAsCBBYCAwECHgcCF4AACgkQXBt0oWUcX2hbCQEAtru7kvM8hi8zo6z9ux2h K+B5xViKuo7Z8K3IXuK5ugwA+wUfKzomzdBPhfxDsqLcEziGRxoyx0Q3ld9aermBUccHtBxCasO2 cm4gQmlkYXIgPG1lQHRoYW9kYW4uZGU+iJMEExYKADsCGwMFCwkIBwICIgIGFQoJCAsCBBYCAwEC HgcCF4AWIQRR8XbrdUcwHqcpqKtcG3ShZRxfaAUCZNf2FQAKCRBcG3ShZRxfaCzSAP4hZ7cSp0YN XYpcjHdsySh2MuBhhoPeLGXs+2kSiqBiOwD/TP8AgPEg/R+SI9GI9on7fBJJ0mp2IT8kZ2rhDOjg gA6IkwQTFgoAOxYhBFHxdut1RzAepymoq1wbdKFlH X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:295608 Archived-At: Eli Zaretskii writes: >> From: Bj=C3=B6rn Bidar >> Cc: Stefan Kangas , luangruo@yahoo.com, >> 74413@debbugs.gnu.org >> Date: Mon, 18 Nov 2024 18:54:26 +0200 >>=20 >> Eli Zaretskii writes: >>=20 >> > This is already available if Emacs is built in a Git repository, and >> > the information is stored in the dumped Emacs. So what is gained by >> > also recording the repository version on a disk file external to >> > Emacs? >>=20 >> The information is only stored if the worker already had git installed >> and checked out the sources with git inside the worker. >> Also the function currently fails unless the system the user uses also h= appens to >> have git installed and the sources if the are installed also contain the >> VCS metadata. >> Storing the VCS metadata in the sources doesn't happen usually as it >> increases the size of a good chunk. In my case e.g. from 188MB to 788MB. >> Why not have the same feature for other platforms too? > > Sorry, I don't understand this explanation; I'm probably missing > something. > > The feature you propose requires to build Emacs inside a Git > repository, is that correct? Because otherwise "git rev-parse" will > not work, right? If that is correct, then building Emacs inside a Git > repository already calls this Git command and records the result in 2 > Emacs Lisp variables. > > So why do you also want to record the same > information on a file? What kind of scenario do you have in mind in > which building Emacs with its current code will not record the branch > and the revision, but your additions to Makefile will record that? The additions to the make file are so that if the worker contains git the file can be generated so that the related functions will still work after the built or to generate them prior the built. The latter probably makes less sense except to maybe avoid having autotools in the built dependency chain. If the Make recipe isn't used to generate the version file it can be generated by the CI, e.g. in my case I take the information from the open build source service. For others such as Fedora the sources can be retrieved in a similar manner. The file can be added before the built starts and package so that the pdump will contain the repository information and the VCS function will also work afterwards.=20