From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Tony Zorman Newsgroups: gmane.emacs.devel Subject: Re: package-vc support for :files keyword Date: Mon, 18 Sep 2023 21:40:13 +0200 Message-ID: <87o7hzi82q.fsf@hyperspace> References: <87ttrshrib.fsf@hyperspace> <871qev6e53.fsf@posteo.net> <87r0mviltd.fsf@hyperspace> <871qevbhru.fsf@posteo.net> 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="33792"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org To: Philip Kaludercic Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Tue Sep 19 04:22:28 2023 Return-path: Envelope-to: ged-emacs-devel@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 1qiQNg-0008bJ-0u for ged-emacs-devel@m.gmane-mx.org; Tue, 19 Sep 2023 04:22:28 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qiQMe-0004jN-Gv; Mon, 18 Sep 2023 22:21:24 -0400 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 1qiK6e-0004Ww-97 for emacs-devel@gnu.org; Mon, 18 Sep 2023 15:40:29 -0400 Original-Received: from mout-p-101.mailbox.org ([80.241.56.151]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_CHACHA20_POLY1305:256) (Exim 4.90_1) (envelope-from ) id 1qiK6W-0006Ug-Sc for emacs-devel@gnu.org; Mon, 18 Sep 2023 15:40:24 -0400 Original-Received: from smtp2.mailbox.org (smtp2.mailbox.org [10.196.197.2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-101.mailbox.org (Postfix) with ESMTPS id 4RqFWP0HRVz9smr; Mon, 18 Sep 2023 21:40:17 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mailbox.org; s=mail20150812; t=1695066017; h=from:from: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=YRCD2Wdrzjt99dV2gug6oj6Ab0t1v1dFG+L1iibJOss=; b=LOqGpDH6IqDFmanDzBtJ5htt4OsQh3BH3vwOUPW39KCyr8KOOB/Gn4Wk0c8yQwXNp/d5N6 GpImd0DGwXl+rYRI+4A43/+D2S1z7HKTZ2ZwpPg4lT9a710XO/01LAV22VbG0GUBCnFcLU 0MEiULxGoyT+8WlDt/clYFxC0EMAhxQiSoIOjBYzNpNZ0KVzEex3zXNI/LOa3pAwwJNShO UfKh6FdkMc0gwHYdETUNjxklq6Lo2bLOR+AWJsqY1dElayvFtjsnyCK49j3A4Mb12i445y 2neGf8tKNSBLqqP2zCqcbp3xjP8jFdXk2X581m0a96G2zX0oFn6EElvrs/lbdw== In-Reply-To: <871qevbhru.fsf@posteo.net> X-MBO-RS-META: 4a9zmeay8zz68x81bpinaaecirar51j9 X-MBO-RS-ID: 7c3d1c2e9b199d00406 Received-SPF: pass client-ip=80.241.56.151; envelope-from=tonyzorman@mailbox.org; helo=mout-p-101.mailbox.org X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-Mailman-Approved-At: Mon, 18 Sep 2023 22:21:22 -0400 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:310732 Archived-At: On Mon, Sep 18 2023 15:52, Philip Kaludercic wrote: >> I think that >> >> $ rm foo.el >> $ git update-index --assume-unchanged foo.el >> >> should work. It should merge cleanly (I've tried this out just now, and >> it worked, but I may have overlooked something). If it's part of the >> package description, then updating should work out of the box, since >> package-vc-upgrade also calls package-vc--unpack-1, which would execute >> the respective :early-shell-command again. > > Being a git-specific command, this shouldn't be added to package-vc > directly. If there is a VCS agnostic/generalisable way of doing this, > then it could be added to VC. Ah, sometimes I forget that there are VCs other than Git=E2=80=94sorry :) > But for now, if I understand you correctly, you are suggesting that > users give package specifications like this: > > (foo :url "https://some.vcs/repository" > ;; ... > :early-shell-command "rm [all the files]; git update-index --assume-= unchanged [all the files]") > > where [all the files] might change between updates. Yes, exactly. > At this point I continue to question the utility of emulating > MELPA-style :files attributes, unless there are concrete usability > issues. > > For the record, these are all the repositories in {Non,}GNU ELPA that > develop multiple packages in a single repository: > > [=E2=80=A6 12 lines elided =E2=80=A6] > > From what I understand, there is no technical necessity for this mode of > development? I wonder how difficult it would be to push for a > one-package-one-repo approach. This is not just for multiple packages in a single repository=E2=80=94at le= ast one has to somewhat broaden what "multiple packages" means. Some packages include small shims for bigger projects, and inadvertently require them as dependencies. The original issue[1] on the vc-use-package repo mentions org-ql[2], more specifically its helm integration in the form of helm-org-ql.el. Some people might not want to pull down helm as a dependency just for one file that they are not going to use anyways. I'm not sure how common of a situation this actually is, but at least for the big completion frameworks=E2=80=94helm and ivy=E2=80=94it's not tot= ally unheard of. [1]: https://github.com/slotThe/vc-use-package/issues/12 [2]: https://github.com/alphapapa/org-ql/ --=20 Tony Zorman | https://tony-zorman.com/