diff mbox series

[bug#71793] ui: Write down the unwritten rule.

Message ID a2ddbb6ee340a4b9fd1e638aed2827d045011aca.1719488650.git.~@wolfsden.cz
State New
Headers show
Series [bug#71793] ui: Write down the unwritten rule. | expand

Commit Message

Tomas Volf June 27, 2024, 11:44 a.m. UTC 
Quote from Ludovic:

> [..] there’s the unwritten rule that non-UI modules should not load (guix
> ui) [..]

This commit turns it into a written rule.

* guix/ui.scm: Document the import policy.

Change-Id: Iedd48f5622b2b17b9e2eec817cf27f0c3a938ff2
---
 guix/ui.scm | 4 ++++
 1 file changed, 4 insertions(+)

Comments

jgart June 29, 2024, 6:47 a.m. UTC | #1 
> Quote from Ludovic:
> 

Hi Tomas,

Can you link me to the places where you are quoting that from? Thanks!

all best,

jgart
Tomas Volf June 29, 2024, 6:43 p.m. UTC | #2 
On 2024-06-29 01:47:52 -0500, jgart wrote:
>
> > Quote from Ludovic:
> >
>
> Hi Tomas,
>
> Can you link me to the places where you are quoting that from? Thanks!

Yes sure, it was given as a reason to not merge patch #2 in 70839:
  https://issues.guix.gnu.org/70839#4-lineno17

It is always possible I just misunderstood.

Have a nice day,
Tomas Volf

--
There are only two hard things in Computer Science:
cache invalidation, naming things and off-by-one errors.
diff mbox series

Patch

diff --git a/guix/ui.scm b/guix/ui.scm
index d82fa533cc..d24886d156 100644
--- a/guix/ui.scm
+++ b/guix/ui.scm
@@ -35,6 +35,10 @@ 
 ;;; You should have received a copy of the GNU General Public License
 ;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
 
+;;; !!!!
+;;; !!!! This module shall not be imported from non-UI modules.
+;;; !!!!
+
 (define-module (guix ui)
   #:use-module (guix i18n)
   #:use-module (guix colors)