Bug 249273

Summary: pkgbase: missing colon in /usr/src/release/packages/runtime.ucl
Product: Base System Reporter: Martin <martin.jakob>
Component: miscAssignee: Kyle Evans <kevans>
Status: Closed FIXED    
Severity: Affects Only Me CC: allanjude, kevans, lwhsu, pkgbase
Priority: --- Keywords: patch, pkgbase
Version: CURRENT   
Hardware: Any   
OS: Any   

Description Martin 2020-09-12 08:36:35 UTC
There is a colon missing in the /usr/src/release/packages/runtime.ucl file, see below.

--- /usr/src/release/packages/runtime.ucl       2019-06-02 09:09:55.106842000 +0200
+++ runtime.ucl 2020-09-12 09:21:57.013004000 +0200
@@ -30,7 +30,7 @@
         version: "%VERSION%"
     }
 }
-directories {
+directories: {
     /dev = "y";
     /tmp = "y";
 }
Comment 1 Allan Jude freebsd_committer freebsd_triage 2020-09-18 13:46:01 UTC
It is not required

As explained here:

https://github.com/vstakhov/libucl/#general-syntax-sugar

key = value;
section {
    key = value;
}

is equivalent to

{
    "key": "value",
    "section": {
        "key": "value"
    }
}


However, it would likely be stylistically better to be consistent throughout the file.
Comment 2 commit-hook freebsd_committer freebsd_triage 2020-09-18 14:29:17 UTC
A commit references this bug:

Author: kevans
Date: Fri Sep 18 14:29:02 UTC 2020
New revision: 365883
URL: https://svnweb.freebsd.org/changeset/base/365883

Log:
  pkgbase: use consistent annotation for objectk eys

  Everywhere else we use objects ("scripts", generally) we do sepcify the
  optional colon. Be consistent and do the same for directories.

  PR:		249273
  Submitted by:	Martin <martin.jakob gmx com>
  MFC after:	1 week

Changes:
  head/release/packages/runtime.ucl
Comment 3 Martin 2020-10-11 08:19:31 UTC
Thanks, i like consistency :)

Can i close the ticket, or does this happen automatically?
Comment 4 Kyle Evans freebsd_committer freebsd_triage 2020-10-11 12:16:08 UTC
Whoops. =)
Comment 5 commit-hook freebsd_committer freebsd_triage 2020-11-14 20:44:42 UTC
A commit references this bug:

Author: kevans
Date: Sat Nov 14 20:44:34 UTC 2020
New revision: 367698
URL: https://svnweb.freebsd.org/changeset/base/367698

Log:
  MFC r365883: pkgbase: use consistent annotation for object keys

  Everywhere else we use objects ("scripts", generally) we do specify the
  optional colon. Be consistent and do the same for directories.

  PR:		249273

Changes:
_U  stable/12/
  stable/12/release/packages/runtime.ucl