From owner-dev-commits-ports-main@freebsd.org Thu Apr 29 05:48:08 2021 Return-Path: Delivered-To: dev-commits-ports-main@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 078CF5F7F43 for ; Thu, 29 Apr 2021 05:48:08 +0000 (UTC) (envelope-from decke@bluelife.at) Received: from mail-lj1-x230.google.com (mail-lj1-x230.google.com [IPv6:2a00:1450:4864:20::230]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4FW4Kf72xQz3t5w for ; Thu, 29 Apr 2021 05:48:06 +0000 (UTC) (envelope-from decke@bluelife.at) Received: by mail-lj1-x230.google.com with SMTP id z23so28494292lji.4 for ; Wed, 28 Apr 2021 22:48:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bluelife.at; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=6FBzIgDLdOZsNNoqL1PY3YjSfRzWUFBlhw+rcUMXQ/U=; b=BK+YiMiDNrMWpt82zCtkPgUTHqqGxp64bfE82BntlmzkylkWJep3TuBprmxex5OGN1 vi/4nFW88/prVPcKO6pfJd68jn4r4XTgE5mZWkFIJ0Z8Pfp5vS3FPg1/LiKjbi4tx18t DtUnHPdtZJ9ggtUdQJM7Jh/bbhrylkemx14pg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=6FBzIgDLdOZsNNoqL1PY3YjSfRzWUFBlhw+rcUMXQ/U=; b=TLVJrgwOH9M7l9ikUo0S6u/wyKG4afqUUQhcIZXMruQ2zLb1Fpux8BKccaqQo2SmX/ Ucl3cqk1DKhvp3k6L3rt4doioI7Al7nEJHw/xKXCL9UEY50tWA3u4E+/Hwg37zVX64pa A2CIEV7AnbM1KfoeKzDpTyijJILSIhoJPrPjLRD8Nnzprz65d2LlIEI4P1TPfm61Ek3R pnKr0pokObOPG4l30GOGp778Tk3DxW6uI+YGh6JW/nyH2PdT2LwPnrDbeZ2Aoj0QyKWJ 9yKuk4e4yqepZvapZxEiNZysIL0CZP6a4+hRgldnbLFSP7oin9U2AaPEZmiHzDVUhGlO 4ulQ== X-Gm-Message-State: AOAM530eO+7TWUz4mBSryU4R2kAQ+eLXNb5ObdTkqjbAZBXmQaD5O4eh r3auSfmAzPFSEay6R2yiUGu5O1MF6mhIrDG4s1BSlw== X-Google-Smtp-Source: ABdhPJzQwISYDlhfGZ6wAbdDQzPizNlZU8AS0uVEnRVqXLtl9bsn70ayXWWsGgYcFzEn3Fbdn7ZQ3dyv3cz87iHOwAE= X-Received: by 2002:a2e:818b:: with SMTP id e11mr17975093ljg.468.1619675284203; Wed, 28 Apr 2021 22:48:04 -0700 (PDT) MIME-Version: 1.0 References: <202104281017.13SAHDF2019723@gitrepo.freebsd.org> <2794032.slGk94SIus@beastie.bionicmutton.org> In-Reply-To: From: =?UTF-8?Q?Bernhard_Fr=C3=B6hlich?= Date: Thu, 29 Apr 2021 07:47:48 +0200 Message-ID: Subject: Re: Please use conventional commit messages (Re: git: 2a9b7ca2a362) To: Alexey Dokuchaev Cc: Rene Ladan , Adriaan de Groot , ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 4FW4Kf72xQz3t5w X-Spamd-Bar: --- Authentication-Results: mx1.freebsd.org; dkim=pass header.d=bluelife.at header.s=google header.b=BK+YiMiD; dmarc=pass (policy=none) header.from=bluelife.at; spf=pass (mx1.freebsd.org: domain of decke@bluelife.at designates 2a00:1450:4864:20::230 as permitted sender) smtp.mailfrom=decke@bluelife.at X-Spamd-Result: default: False [-4.00 / 15.00]; RCVD_TLS_ALL(0.00)[]; ARC_NA(0.00)[]; R_DKIM_ALLOW(-0.20)[bluelife.at:s=google]; FREEFALL_USER(0.00)[decke]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; R_SPF_ALLOW(-0.20)[+ip6:2a00:1450:4000::/36]; NEURAL_HAM_LONG(-1.00)[-1.000]; MIME_GOOD(-0.10)[text/plain]; PREVIOUSLY_DELIVERED(0.00)[dev-commits-ports-main@freebsd.org]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; RCPT_COUNT_FIVE(0.00)[6]; SPAMHAUS_ZRD(0.00)[2a00:1450:4864:20::230:from:127.0.2.255]; TO_MATCH_ENVRCPT_SOME(0.00)[]; DKIM_TRACE(0.00)[bluelife.at:+]; DMARC_POLICY_ALLOW(-0.50)[bluelife.at,none]; RCVD_IN_DNSWL_NONE(0.00)[2a00:1450:4864:20::230:from]; NEURAL_HAM_SHORT(-1.00)[-1.000]; FROM_EQ_ENVFROM(0.00)[]; MIME_TRACE(0.00)[0:+]; RBL_DBL_DONT_QUERY_IPS(0.00)[2a00:1450:4864:20::230:from]; ASN(0.00)[asn:15169, ipnet:2a00:1450::/32, country:US]; RCVD_COUNT_TWO(0.00)[2]; MAILMAN_DEST(0.00)[dev-commits-ports-main] X-BeenThere: dev-commits-ports-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commits to the main branch of the FreeBSD ports repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 29 Apr 2021 05:48:08 -0000 On Thu, Apr 29, 2021 at 6:10 AM Alexey Dokuchaev wrote: > > On Wed, Apr 28, 2021 at 12:44:24PM +0000, Rene Ladan wrote: > > On Wed, Apr 28, 2021 at 11:53:46AM +0000, Alexey Dokuchaev wrote: > > > On Wed, Apr 28, 2021 at 12:36:39PM +0200, Adriaan de Groot wrote: > > > > On Wednesday, 28 April 2021 12:17:13 CEST Alexey Dokuchaev wrote: > > > > > commit 2a9b7ca2a3620249bfad050ac3eee6d859926213 > > > > > ... > > > > > > > > > > Update `www/web2ldap' to version 1.5.117. > > > > > > > > Please use conventional commit messages, [...] > > Forgot to mention: this "conventional" format of yours adds really a lot > of noise to the "git log ." output which in turn makes history harder to > study and navigate around. Extra blank lines pessimize it even further. > > > > I think it is conventional enough. I'd prefer it even better without > > > the embedded port name, but I guess I can buy the reasoning why peopl= e > > > want to see it there. As for its position, I find "foo/bar: blah" > > > inconvenient, unnatural, and hard to read. > > > > This just makes scanning commit mails harder. > > The problem here is very poor git's default commit email template, and > duplicating metainformation that can be automatically inferred from the > commit is conceptually wrong. > > Subject line should contain commit identifier, branch name, and a list > of files, directories, or the top directory of the commit. Subversion > did this right, FWIW. Take a look at old commit logs in comparison and > notice how cleaner and better they were. > > Li-Wen told me that they are working on improving email template (as it > has other deficiencies as well), so let's hope this issue would be fixed > at the proper level and people won't have to write idiotic subject-like > lines in their commit logs. I know your feelings about git and yeah it might not be the best tool one could possible think of but let's be positive and make the best out of it. There might be some room for improvements, yeah I would also like to see the commit message templates to be improved and a few nits here and there but writing a good commit message will ALWAYS be necessary so let's focus on that for the moment. If my mailbox in the morning contains 200 messages and I just want to scan them if something interesting happened then I don't read each and every subject in full but I just want to see if my eye finds a port that I care a= bout or stuff that I find interesting. It's about the same situation why newspap= ers have headlines. It tries to use keywords to get your attention. In git the "short logs" are very compact: https://cgit.freebsd.org/ports/log/?ofs=3D50 whereas in svn traditionally you always get the full newspaper: https://svnweb.freebsd.org/ports/head/?view=3Dlog A common format that everyone uses is immensely useful in this case because it allows scanning many commits quite fast. Many people have used that format also before the git conversion already and it has proven t= o be useful for many of us. I know that I have no right to expect anything from other peoples commit messages but you just might not have this usecase so probably it helps you understand other people better if you know their usecases. And maybe you also realize that it sometimes helps a community as a whole to stick to some common tools and formats after everyone has expressed why a new tool/format is imperfect. --=20 Bernhard Fr=C3=B6hlich http://www.bluelife.at/