* [PATCH v2] userdiff: Add a builtin pattern for dts files
@ 2019-08-16 22:56 Stephen Boyd
2019-08-17 1:48 ` Frank Rowand
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Stephen Boyd @ 2019-08-16 22:56 UTC (permalink / raw)
To: git
Cc: Adrian Johnson, William Duclot, Johannes Sixt, Matthieu Moy,
devicetree, Alban Gruin, Junio C Hamano, Rob Herring
The Linux kernel receives many patches to the devicetree files each
release. The hunk header for those patches typically show nothing,
making it difficult to figure out what node is being modified without
applying the patch or opening the file and seeking to the context. Let's
add a builtin 'dts' pattern to git so that users can get better diff
output on dts files when they use the diff=dts driver.
The regex has been constructed based on the spec at devicetree.org[1]
[1] https://github.com/devicetree-org/devicetree-specification/releases/latest
Cc: Rob Herring <robh+dt@kernel.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
---
Sending this again after getting feedback and it getting stuck in
review[1]. I'm not sure what happened with the meta question from Junio
to add a way for various projects to introduce their own patterns, but
I'd still prefer to have this in git proper because the kernel uses git
extensively and we rely on git formatted patches in our workflow. I
recently reviewed a dts change and remembered this never got accepted.
Changes from v1:
* Updated regex to handle anything after node names instead of
requiring a '{'
* Updated test for boolean relation operators
* Sent out a patch to devicetree spec to document % operator
[1] Feedback was in 16335abe-5e7e-fd7a-25f4-373f94e176e1@gmail.com
Documentation/gitattributes.txt | 2 ++
t/t4018-diff-funcname.sh | 1 +
t/t4018/dts-labels | 8 +++++++
t/t4018/dts-node-unitless | 8 +++++++
t/t4018/dts-nodes | 8 +++++++
t/t4018/dts-reference | 8 +++++++
t/t4034-diff-words.sh | 1 +
t/t4034/dts/expect | 37 +++++++++++++++++++++++++++++++++
t/t4034/dts/post | 32 ++++++++++++++++++++++++++++
t/t4034/dts/pre | 32 ++++++++++++++++++++++++++++
userdiff.c | 9 ++++++++
11 files changed, 146 insertions(+)
create mode 100644 t/t4018/dts-labels
create mode 100644 t/t4018/dts-node-unitless
create mode 100644 t/t4018/dts-nodes
create mode 100644 t/t4018/dts-reference
create mode 100644 t/t4034/dts/expect
create mode 100644 t/t4034/dts/post
create mode 100644 t/t4034/dts/pre
diff --git a/Documentation/gitattributes.txt b/Documentation/gitattributes.txt
index fb1d188d440c..c5a528c667b6 100644
--- a/Documentation/gitattributes.txt
+++ b/Documentation/gitattributes.txt
@@ -810,6 +810,8 @@ patterns are available:
- `css` suitable for cascading style sheets.
+- `dts` suitable for devicetree (DTS) files.
+
- `fortran` suitable for source code in the Fortran language.
- `fountain` suitable for Fountain documents.
diff --git a/t/t4018-diff-funcname.sh b/t/t4018-diff-funcname.sh
index 9261d6d3a000..6f5ef0035e92 100755
--- a/t/t4018-diff-funcname.sh
+++ b/t/t4018-diff-funcname.sh
@@ -31,6 +31,7 @@ diffpatterns="
cpp
csharp
css
+ dts
fortran
fountain
golang
diff --git a/t/t4018/dts-labels b/t/t4018/dts-labels
new file mode 100644
index 000000000000..27cd4921cfb6
--- /dev/null
+++ b/t/t4018/dts-labels
@@ -0,0 +1,8 @@
+/ {
+ label_1: node1@ff00 {
+ label2: RIGHT {
+ vendor,some-property;
+ ChangeMe = <0x45-30>;
+ };
+ };
+};
diff --git a/t/t4018/dts-node-unitless b/t/t4018/dts-node-unitless
new file mode 100644
index 000000000000..c5287d91416e
--- /dev/null
+++ b/t/t4018/dts-node-unitless
@@ -0,0 +1,8 @@
+/ {
+ label_1: node1 {
+ RIGHT {
+ prop-array = <1>, <4>;
+ ChangeMe = <0xffeedd00>;
+ };
+ };
+};
diff --git a/t/t4018/dts-nodes b/t/t4018/dts-nodes
new file mode 100644
index 000000000000..5a4334bb1645
--- /dev/null
+++ b/t/t4018/dts-nodes
@@ -0,0 +1,8 @@
+/ {
+ label_1: node1@ff00 {
+ RIGHT@deadf00,4000 {
+ #size-cells = <1>;
+ ChangeMe = <0xffeedd00>;
+ };
+ };
+};
diff --git a/t/t4018/dts-reference b/t/t4018/dts-reference
new file mode 100644
index 000000000000..f115d4291d25
--- /dev/null
+++ b/t/t4018/dts-reference
@@ -0,0 +1,8 @@
+&label_1 {
+ TEST = <455>;
+};
+
+&RIGHT {
+ vendor,some-property;
+ ChangeMe = <0x45-30>;
+};
diff --git a/t/t4034-diff-words.sh b/t/t4034-diff-words.sh
index 912df91226f2..9a93c2a3e0dd 100755
--- a/t/t4034-diff-words.sh
+++ b/t/t4034-diff-words.sh
@@ -303,6 +303,7 @@ test_language_driver bibtex
test_language_driver cpp
test_language_driver csharp
test_language_driver css
+test_language_driver dts
test_language_driver fortran
test_language_driver html
test_language_driver java
diff --git a/t/t4034/dts/expect b/t/t4034/dts/expect
new file mode 100644
index 000000000000..560fc9918476
--- /dev/null
+++ b/t/t4034/dts/expect
@@ -0,0 +1,37 @@
+<BOLD>diff --git a/pre b/post<RESET>
+<BOLD>index b6a9051..7803aee 100644<RESET>
+<BOLD>--- a/pre<RESET>
+<BOLD>+++ b/post<RESET>
+<CYAN>@@ -1,32 +1,32 @@<RESET>
+/ {<RESET>
+ <RED>this_handle<RESET><GREEN>HANDLE_2<RESET>: <RED>node<RESET><GREEN>new-node<RESET>@<RED>f00<RESET><GREEN>eeda<RESET> {
+ compatible = "<RED>mydev<RESET><GREEN>vendor,compat<RESET>";
+ string-prop = <RED>start<RESET><GREEN>end<RESET>: "hello <RED>world!<RESET><GREEN>world?<RESET>" <RED>end<RESET><GREEN>start<RESET>: ;
+ <RED>#size-cells<RESET><GREEN>#address-cells<RESET> = <<RED>0+0<RESET><GREEN>0+40<RESET>>;
+ reg = <<RED>0xf00<RESET><GREEN>0xeeda<RESET>>;
+ prop = <<GREEN>(<RESET>1<GREEN>)<RESET>>;
+ prop = <<GREEN>(<RESET>-1e10<GREEN>)<RESET>>;
+ prop = <(!<RED>3<RESET><GREEN>1<RESET>)>;
+ prop = <(~<RED>3<RESET><GREEN>1<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>/<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>%<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3+4<RESET><GREEN>1+2<RESET>)>;
+ prop = <(<RED>3-4<RESET><GREEN>1-2<RESET>)>;
+ prop = /bits/ <RED>64<RESET><GREEN>32<RESET> <(<RED>3<RESET><GREEN>1<RESET><<<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>>><RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>^<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>|<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>&&<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>3<RESET><GREEN>1<RESET>||<RED>4<RESET><GREEN>2<RESET>)>;
+ prop = <(<RED>4?5<RESET><GREEN>1?2<RESET>:3)>;
+ list = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>, <0 0 0 <RED>0<RESET><GREEN>1<RESET>>;
+ };<RESET>
+
+ &<RED>phandle<RESET><GREEN>phandle2<RESET> {
+ <RED>pre-phandle<RESET><GREEN>prop_handle<RESET> = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>;
+ };<RESET>
+};<RESET>
diff --git a/t/t4034/dts/post b/t/t4034/dts/post
new file mode 100644
index 000000000000..7803aee28093
--- /dev/null
+++ b/t/t4034/dts/post
@@ -0,0 +1,32 @@
+/ {
+ HANDLE_2: new-node@eeda {
+ compatible = "vendor,compat";
+ string-prop = end: "hello world?" start: ;
+ #address-cells = <0+40>;
+ reg = <0xeeda>;
+ prop = <(1)>;
+ prop = <(-1e10)>;
+ prop = <(!1)>;
+ prop = <(~1)>;
+ prop = <(1*2)>;
+ prop = <(1&2)>;
+ prop = <(1*2)>;
+ prop = <(1/2)>;
+ prop = <(1%2)>;
+ prop = <(1+2)>;
+ prop = <(1-2)>;
+ prop = /bits/ 32 <(1<<2)>;
+ prop = <(1>>2)>;
+ prop = <(1&2)>;
+ prop = <(1^2)>;
+ prop = <(1|2)>;
+ prop = <(1&&2)>;
+ prop = <(1||2)>;
+ prop = <(1?2:3)>;
+ list = <&HANDLE_2>, <0 0 0 1>;
+ };
+
+ &phandle2 {
+ prop_handle = <&HANDLE_2>;
+ };
+};
diff --git a/t/t4034/dts/pre b/t/t4034/dts/pre
new file mode 100644
index 000000000000..b6a905113c22
--- /dev/null
+++ b/t/t4034/dts/pre
@@ -0,0 +1,32 @@
+/ {
+ this_handle: node@f00 {
+ compatible = "mydev";
+ string-prop = start: "hello world!" end: ;
+ #size-cells = <0+0>;
+ reg = <0xf00>;
+ prop = <1>;
+ prop = <-1e10>;
+ prop = <(!3)>;
+ prop = <(~3)>;
+ prop = <(3*4)>;
+ prop = <(3&4)>;
+ prop = <(3*4)>;
+ prop = <(3/4)>;
+ prop = <(3%4)>;
+ prop = <(3+4)>;
+ prop = <(3-4)>;
+ prop = /bits/ 64 <(3<<4)>;
+ prop = <(3>>4)>;
+ prop = <(3&4)>;
+ prop = <(3^4)>;
+ prop = <(3|4)>;
+ prop = <(3&&4)>;
+ prop = <(3||4)>;
+ prop = <(4?5:3)>;
+ list = <&this_handle>, <0 0 0 0>;
+ };
+
+ &phandle {
+ pre-phandle = <&this_handle>;
+ };
+};
diff --git a/userdiff.c b/userdiff.c
index e74a6d402255..1db5d30aaebe 100644
--- a/userdiff.c
+++ b/userdiff.c
@@ -23,6 +23,15 @@ IPATTERN("ada",
"[a-zA-Z][a-zA-Z0-9_]*"
"|[-+]?[0-9][0-9#_.aAbBcCdDeEfF]*([eE][+-]?[0-9_]+)?"
"|=>|\\.\\.|\\*\\*|:=|/=|>=|<=|<<|>>|<>"),
+PATTERNS("dts",
+ /* Node name with optional label and unit address */
+ "^[ \t]*((([a-zA-Z_][a-zA-Z0-9_]*:[ \t]*)?[a-zA-Z][a-zA-Z0-9,._+-]*(@[a-zA-Z0-9,._+-]+)?"
+ /* Reference */
+ "|&[a-zA-Z_][a-zA-Z0-9_]*)[ \t]*[^;]*)$",
+ /* -- */
+ /* Property names and math operators */
+ "[a-zA-Z0-9,._+?#-]+"
+ "|[-+*/%&^|!~]|>>|<<|&&|\\|\\|"),
IPATTERN("fortran",
"!^([C*]|[ \t]*!)\n"
"!^[ \t]*MODULE[ \t]+PROCEDURE[ \t]\n"
--
Sent by a computer through tubes
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH v2] userdiff: Add a builtin pattern for dts files
2019-08-16 22:56 [PATCH v2] userdiff: Add a builtin pattern for dts files Stephen Boyd
@ 2019-08-17 1:48 ` Frank Rowand
2019-08-17 15:18 ` Alban Gruin
2019-08-19 18:40 ` Johannes Sixt
2 siblings, 0 replies; 6+ messages in thread
From: Frank Rowand @ 2019-08-17 1:48 UTC (permalink / raw)
To: Stephen Boyd, git
Cc: Adrian Johnson, William Duclot, Johannes Sixt, Matthieu Moy,
devicetree, Alban Gruin, Junio C Hamano, Rob Herring,
Frank Rowand
+Frank (me)
On 8/16/19 3:56 PM, Stephen Boyd wrote:
> The Linux kernel receives many patches to the devicetree files each
> release. The hunk header for those patches typically show nothing,
> making it difficult to figure out what node is being modified without
> applying the patch or opening the file and seeking to the context. Let's
> add a builtin 'dts' pattern to git so that users can get better diff
> output on dts files when they use the diff=dts driver.
>
> The regex has been constructed based on the spec at devicetree.org[1]
>
> [1] https://github.com/devicetree-org/devicetree-specification/releases/latest
>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> ---
>
> Sending this again after getting feedback and it getting stuck in
> review[1]. I'm not sure what happened with the meta question from Junio
> to add a way for various projects to introduce their own patterns, but
> I'd still prefer to have this in git proper because the kernel uses git
> extensively and we rely on git formatted patches in our workflow. I
> recently reviewed a dts change and remembered this never got accepted.
>
> Changes from v1:
> * Updated regex to handle anything after node names instead of
> requiring a '{'
> * Updated test for boolean relation operators
> * Sent out a patch to devicetree spec to document % operator
>
> [1] Feedback was in 16335abe-5e7e-fd7a-25f4-373f94e176e1@gmail.com
>
> Documentation/gitattributes.txt | 2 ++
> t/t4018-diff-funcname.sh | 1 +
> t/t4018/dts-labels | 8 +++++++
> t/t4018/dts-node-unitless | 8 +++++++
> t/t4018/dts-nodes | 8 +++++++
> t/t4018/dts-reference | 8 +++++++
> t/t4034-diff-words.sh | 1 +
> t/t4034/dts/expect | 37 +++++++++++++++++++++++++++++++++
> t/t4034/dts/post | 32 ++++++++++++++++++++++++++++
> t/t4034/dts/pre | 32 ++++++++++++++++++++++++++++
> userdiff.c | 9 ++++++++
> 11 files changed, 146 insertions(+)
> create mode 100644 t/t4018/dts-labels
> create mode 100644 t/t4018/dts-node-unitless
> create mode 100644 t/t4018/dts-nodes
> create mode 100644 t/t4018/dts-reference
> create mode 100644 t/t4034/dts/expect
> create mode 100644 t/t4034/dts/post
> create mode 100644 t/t4034/dts/pre
>
> diff --git a/Documentation/gitattributes.txt b/Documentation/gitattributes.txt
> index fb1d188d440c..c5a528c667b6 100644
> --- a/Documentation/gitattributes.txt
> +++ b/Documentation/gitattributes.txt
> @@ -810,6 +810,8 @@ patterns are available:
>
> - `css` suitable for cascading style sheets.
>
> +- `dts` suitable for devicetree (DTS) files.
> +
> - `fortran` suitable for source code in the Fortran language.
>
> - `fountain` suitable for Fountain documents.
> diff --git a/t/t4018-diff-funcname.sh b/t/t4018-diff-funcname.sh
> index 9261d6d3a000..6f5ef0035e92 100755
> --- a/t/t4018-diff-funcname.sh
> +++ b/t/t4018-diff-funcname.sh
> @@ -31,6 +31,7 @@ diffpatterns="
> cpp
> csharp
> css
> + dts
> fortran
> fountain
> golang
> diff --git a/t/t4018/dts-labels b/t/t4018/dts-labels
> new file mode 100644
> index 000000000000..27cd4921cfb6
> --- /dev/null
> +++ b/t/t4018/dts-labels
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1@ff00 {
> + label2: RIGHT {
> + vendor,some-property;
> + ChangeMe = <0x45-30>;
> + };
> + };
> +};
> diff --git a/t/t4018/dts-node-unitless b/t/t4018/dts-node-unitless
> new file mode 100644
> index 000000000000..c5287d91416e
> --- /dev/null
> +++ b/t/t4018/dts-node-unitless
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1 {
> + RIGHT {
> + prop-array = <1>, <4>;
> + ChangeMe = <0xffeedd00>;
> + };
> + };
> +};
> diff --git a/t/t4018/dts-nodes b/t/t4018/dts-nodes
> new file mode 100644
> index 000000000000..5a4334bb1645
> --- /dev/null
> +++ b/t/t4018/dts-nodes
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1@ff00 {
> + RIGHT@deadf00,4000 {
> + #size-cells = <1>;
> + ChangeMe = <0xffeedd00>;
> + };
> + };
> +};
> diff --git a/t/t4018/dts-reference b/t/t4018/dts-reference
> new file mode 100644
> index 000000000000..f115d4291d25
> --- /dev/null
> +++ b/t/t4018/dts-reference
> @@ -0,0 +1,8 @@
> +&label_1 {
> + TEST = <455>;
> +};
> +
> +&RIGHT {
> + vendor,some-property;
> + ChangeMe = <0x45-30>;
> +};
> diff --git a/t/t4034-diff-words.sh b/t/t4034-diff-words.sh
> index 912df91226f2..9a93c2a3e0dd 100755
> --- a/t/t4034-diff-words.sh
> +++ b/t/t4034-diff-words.sh
> @@ -303,6 +303,7 @@ test_language_driver bibtex
> test_language_driver cpp
> test_language_driver csharp
> test_language_driver css
> +test_language_driver dts
> test_language_driver fortran
> test_language_driver html
> test_language_driver java
> diff --git a/t/t4034/dts/expect b/t/t4034/dts/expect
> new file mode 100644
> index 000000000000..560fc9918476
> --- /dev/null
> +++ b/t/t4034/dts/expect
> @@ -0,0 +1,37 @@
> +<BOLD>diff --git a/pre b/post<RESET>
> +<BOLD>index b6a9051..7803aee 100644<RESET>
> +<BOLD>--- a/pre<RESET>
> +<BOLD>+++ b/post<RESET>
> +<CYAN>@@ -1,32 +1,32 @@<RESET>
> +/ {<RESET>
> + <RED>this_handle<RESET><GREEN>HANDLE_2<RESET>: <RED>node<RESET><GREEN>new-node<RESET>@<RED>f00<RESET><GREEN>eeda<RESET> {
> + compatible = "<RED>mydev<RESET><GREEN>vendor,compat<RESET>";
> + string-prop = <RED>start<RESET><GREEN>end<RESET>: "hello <RED>world!<RESET><GREEN>world?<RESET>" <RED>end<RESET><GREEN>start<RESET>: ;
> + <RED>#size-cells<RESET><GREEN>#address-cells<RESET> = <<RED>0+0<RESET><GREEN>0+40<RESET>>;
> + reg = <<RED>0xf00<RESET><GREEN>0xeeda<RESET>>;
> + prop = <<GREEN>(<RESET>1<GREEN>)<RESET>>;
> + prop = <<GREEN>(<RESET>-1e10<GREEN>)<RESET>>;
> + prop = <(!<RED>3<RESET><GREEN>1<RESET>)>;
> + prop = <(~<RED>3<RESET><GREEN>1<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>/<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>%<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3+4<RESET><GREEN>1+2<RESET>)>;
> + prop = <(<RED>3-4<RESET><GREEN>1-2<RESET>)>;
> + prop = /bits/ <RED>64<RESET><GREEN>32<RESET> <(<RED>3<RESET><GREEN>1<RESET><<<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>>><RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>^<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>|<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>||<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>4?5<RESET><GREEN>1?2<RESET>:3)>;
> + list = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>, <0 0 0 <RED>0<RESET><GREEN>1<RESET>>;
> + };<RESET>
> +
> + &<RED>phandle<RESET><GREEN>phandle2<RESET> {
> + <RED>pre-phandle<RESET><GREEN>prop_handle<RESET> = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>;
> + };<RESET>
> +};<RESET>
> diff --git a/t/t4034/dts/post b/t/t4034/dts/post
> new file mode 100644
> index 000000000000..7803aee28093
> --- /dev/null
> +++ b/t/t4034/dts/post
> @@ -0,0 +1,32 @@
> +/ {
> + HANDLE_2: new-node@eeda {
> + compatible = "vendor,compat";
> + string-prop = end: "hello world?" start: ;
> + #address-cells = <0+40>;
> + reg = <0xeeda>;
> + prop = <(1)>;
> + prop = <(-1e10)>;
> + prop = <(!1)>;
> + prop = <(~1)>;
> + prop = <(1*2)>;
> + prop = <(1&2)>;
> + prop = <(1*2)>;
> + prop = <(1/2)>;
> + prop = <(1%2)>;
> + prop = <(1+2)>;
> + prop = <(1-2)>;
> + prop = /bits/ 32 <(1<<2)>;
> + prop = <(1>>2)>;
> + prop = <(1&2)>;
> + prop = <(1^2)>;
> + prop = <(1|2)>;
> + prop = <(1&&2)>;
> + prop = <(1||2)>;
> + prop = <(1?2:3)>;
> + list = <&HANDLE_2>, <0 0 0 1>;
> + };
> +
> + &phandle2 {
> + prop_handle = <&HANDLE_2>;
> + };
> +};
> diff --git a/t/t4034/dts/pre b/t/t4034/dts/pre
> new file mode 100644
> index 000000000000..b6a905113c22
> --- /dev/null
> +++ b/t/t4034/dts/pre
> @@ -0,0 +1,32 @@
> +/ {
> + this_handle: node@f00 {
> + compatible = "mydev";
> + string-prop = start: "hello world!" end: ;
> + #size-cells = <0+0>;
> + reg = <0xf00>;
> + prop = <1>;
> + prop = <-1e10>;
> + prop = <(!3)>;
> + prop = <(~3)>;
> + prop = <(3*4)>;
> + prop = <(3&4)>;
> + prop = <(3*4)>;
> + prop = <(3/4)>;
> + prop = <(3%4)>;
> + prop = <(3+4)>;
> + prop = <(3-4)>;
> + prop = /bits/ 64 <(3<<4)>;
> + prop = <(3>>4)>;
> + prop = <(3&4)>;
> + prop = <(3^4)>;
> + prop = <(3|4)>;
> + prop = <(3&&4)>;
> + prop = <(3||4)>;
> + prop = <(4?5:3)>;
> + list = <&this_handle>, <0 0 0 0>;
> + };
> +
> + &phandle {
> + pre-phandle = <&this_handle>;
> + };
> +};
> diff --git a/userdiff.c b/userdiff.c
> index e74a6d402255..1db5d30aaebe 100644
> --- a/userdiff.c
> +++ b/userdiff.c
> @@ -23,6 +23,15 @@ IPATTERN("ada",
> "[a-zA-Z][a-zA-Z0-9_]*"
> "|[-+]?[0-9][0-9#_.aAbBcCdDeEfF]*([eE][+-]?[0-9_]+)?"
> "|=>|\\.\\.|\\*\\*|:=|/=|>=|<=|<<|>>|<>"),
> +PATTERNS("dts",
> + /* Node name with optional label and unit address */
> + "^[ \t]*((([a-zA-Z_][a-zA-Z0-9_]*:[ \t]*)?[a-zA-Z][a-zA-Z0-9,._+-]*(@[a-zA-Z0-9,._+-]+)?"
> + /* Reference */
> + "|&[a-zA-Z_][a-zA-Z0-9_]*)[ \t]*[^;]*)$",
> + /* -- */
> + /* Property names and math operators */
> + "[a-zA-Z0-9,._+?#-]+"
> + "|[-+*/%&^|!~]|>>|<<|&&|\\|\\|"),
> IPATTERN("fortran",
> "!^([C*]|[ \t]*!)\n"
> "!^[ \t]*MODULE[ \t]+PROCEDURE[ \t]\n"
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] userdiff: Add a builtin pattern for dts files
2019-08-16 22:56 [PATCH v2] userdiff: Add a builtin pattern for dts files Stephen Boyd
2019-08-17 1:48 ` Frank Rowand
@ 2019-08-17 15:18 ` Alban Gruin
2019-08-19 18:40 ` Johannes Sixt
2 siblings, 0 replies; 6+ messages in thread
From: Alban Gruin @ 2019-08-17 15:18 UTC (permalink / raw)
To: Stephen Boyd, git
Cc: Adrian Johnson, William Duclot, Johannes Sixt, Matthieu Moy,
devicetree, Junio C Hamano, Rob Herring
Hi Stephen,
Le 17/08/2019 à 00:56, Stephen Boyd a écrit :
> The Linux kernel receives many patches to the devicetree files each
> release. The hunk header for those patches typically show nothing,
> making it difficult to figure out what node is being modified without
> applying the patch or opening the file and seeking to the context. Let's
> add a builtin 'dts' pattern to git so that users can get better diff
> output on dts files when they use the diff=dts driver.
>
> The regex has been constructed based on the spec at devicetree.org[1]
>
> [1] https://github.com/devicetree-org/devicetree-specification/releases/latest
>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> ---
>
> Sending this again after getting feedback and it getting stuck in
> review[1]. I'm not sure what happened with the meta question from Junio
> to add a way for various projects to introduce their own patterns,
Nothing happened, AFAIK.
> but
> I'd still prefer to have this in git proper because the kernel uses git
> extensively and we rely on git formatted patches in our workflow. I
> recently reviewed a dts change and remembered this never got accepted.
>
> Changes from v1:
> * Updated regex to handle anything after node names instead of
> requiring a '{'
> * Updated test for boolean relation operators
> * Sent out a patch to devicetree spec to document % operator
>
> [1] Feedback was in 16335abe-5e7e-fd7a-25f4-373f94e176e1@gmail.com
>
> -%<-
This looks good to me.
Cheers,
Alban
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] userdiff: Add a builtin pattern for dts files
2019-08-16 22:56 [PATCH v2] userdiff: Add a builtin pattern for dts files Stephen Boyd
2019-08-17 1:48 ` Frank Rowand
2019-08-17 15:18 ` Alban Gruin
@ 2019-08-19 18:40 ` Johannes Sixt
2019-08-19 20:44 ` Stephen Boyd
2 siblings, 1 reply; 6+ messages in thread
From: Johannes Sixt @ 2019-08-19 18:40 UTC (permalink / raw)
To: Stephen Boyd
Cc: git, Adrian Johnson, William Duclot, Matthieu Moy, devicetree,
Alban Gruin, Junio C Hamano, Rob Herring, Frank Rowand
Am 17.08.19 um 00:56 schrieb Stephen Boyd:
> The Linux kernel receives many patches to the devicetree files each
> release. The hunk header for those patches typically show nothing,
> making it difficult to figure out what node is being modified without
> applying the patch or opening the file and seeking to the context. Let's
> add a builtin 'dts' pattern to git so that users can get better diff
> output on dts files when they use the diff=dts driver.
>
> The regex has been constructed based on the spec at devicetree.org[1]
>
> [1] https://github.com/devicetree-org/devicetree-specification/releases/latest
>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> ---
>
> Sending this again after getting feedback and it getting stuck in
> review[1]. I'm not sure what happened with the meta question from Junio
> to add a way for various projects to introduce their own patterns, but
> I'd still prefer to have this in git proper because the kernel uses git
> extensively and we rely on git formatted patches in our workflow. I
> recently reviewed a dts change and remembered this never got accepted.
>
> Changes from v1:
> * Updated regex to handle anything after node names instead of
> requiring a '{'
> * Updated test for boolean relation operators
> * Sent out a patch to devicetree spec to document % operator
>
> [1] Feedback was in 16335abe-5e7e-fd7a-25f4-373f94e176e1@gmail.com
Thanks. I've a few suggestions below.
> diff --git a/t/t4018/dts-labels b/t/t4018/dts-labels
> new file mode 100644
> index 000000000000..27cd4921cfb6
> --- /dev/null
> +++ b/t/t4018/dts-labels
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1@ff00 {
> + label2: RIGHT {
> + vendor,some-property;
> + ChangeMe = <0x45-30>;
In these tests, it would be worthwhile to leave another (possibly blank)
line before the ChangeMe line in order to demonstrate that lines
beginning with a word, such as the 'vendor,some-property;' line, are
_not_ picked up when they are not in the hunk context.
> + };
> + };
> +};
> diff --git a/t/t4018/dts-node-unitless b/t/t4018/dts-node-unitless
> new file mode 100644
> index 000000000000..c5287d91416e
> --- /dev/null
> +++ b/t/t4018/dts-node-unitless
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1 {
> + RIGHT {
> + prop-array = <1>, <4>;
> + ChangeMe = <0xffeedd00>;
> + };
> + };
> +};
> diff --git a/t/t4018/dts-nodes b/t/t4018/dts-nodes
> new file mode 100644
> index 000000000000..5a4334bb1645
> --- /dev/null
> +++ b/t/t4018/dts-nodes
> @@ -0,0 +1,8 @@
> +/ {
> + label_1: node1@ff00 {
> + RIGHT@deadf00,4000 {
> + #size-cells = <1>;
> + ChangeMe = <0xffeedd00>;
> + };
> + };
> +};
> diff --git a/t/t4018/dts-reference b/t/t4018/dts-reference
> new file mode 100644
> index 000000000000..f115d4291d25
> --- /dev/null
> +++ b/t/t4018/dts-reference
> @@ -0,0 +1,8 @@
> +&label_1 {
> + TEST = <455>;
> +};
> +
> +&RIGHT {
> + vendor,some-property;
> + ChangeMe = <0x45-30>;
> +};
> diff --git a/t/t4034-diff-words.sh b/t/t4034-diff-words.sh
> index 912df91226f2..9a93c2a3e0dd 100755
> --- a/t/t4034-diff-words.sh
> +++ b/t/t4034-diff-words.sh
> @@ -303,6 +303,7 @@ test_language_driver bibtex
> test_language_driver cpp
> test_language_driver csharp
> test_language_driver css
> +test_language_driver dts
> test_language_driver fortran
> test_language_driver html
> test_language_driver java
> diff --git a/t/t4034/dts/expect b/t/t4034/dts/expect
> new file mode 100644
> index 000000000000..560fc9918476
> --- /dev/null
> +++ b/t/t4034/dts/expect
> @@ -0,0 +1,37 @@
> +<BOLD>diff --git a/pre b/post<RESET>
> +<BOLD>index b6a9051..7803aee 100644<RESET>
> +<BOLD>--- a/pre<RESET>
> +<BOLD>+++ b/post<RESET>
> +<CYAN>@@ -1,32 +1,32 @@<RESET>
> +/ {<RESET>
> + <RED>this_handle<RESET><GREEN>HANDLE_2<RESET>: <RED>node<RESET><GREEN>new-node<RESET>@<RED>f00<RESET><GREEN>eeda<RESET> {
> + compatible = "<RED>mydev<RESET><GREEN>vendor,compat<RESET>";
> + string-prop = <RED>start<RESET><GREEN>end<RESET>: "hello <RED>world!<RESET><GREEN>world?<RESET>" <RED>end<RESET><GREEN>start<RESET>: ;
> + <RED>#size-cells<RESET><GREEN>#address-cells<RESET> = <<RED>0+0<RESET><GREEN>0+40<RESET>>;
> + reg = <<RED>0xf00<RESET><GREEN>0xeeda<RESET>>;
> + prop = <<GREEN>(<RESET>1<GREEN>)<RESET>>;
> + prop = <<GREEN>(<RESET>-1e10<GREEN>)<RESET>>;
> + prop = <(!<RED>3<RESET><GREEN>1<RESET>)>;
> + prop = <(~<RED>3<RESET><GREEN>1<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>*<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>/<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>%<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3+4<RESET><GREEN>1+2<RESET>)>;
> + prop = <(<RED>3-4<RESET><GREEN>1-2<RESET>)>;
> + prop = /bits/ <RED>64<RESET><GREEN>32<RESET> <(<RED>3<RESET><GREEN>1<RESET><<<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>>><RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>^<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>|<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>&&<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>3<RESET><GREEN>1<RESET>||<RED>4<RESET><GREEN>2<RESET>)>;
> + prop = <(<RED>4?5<RESET><GREEN>1?2<RESET>:3)>;
> + list = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>, <0 0 0 <RED>0<RESET><GREEN>1<RESET>>;
> + };<RESET>
> +
> + &<RED>phandle<RESET><GREEN>phandle2<RESET> {
> + <RED>pre-phandle<RESET><GREEN>prop_handle<RESET> = <&<RED>this_handle<RESET><GREEN>HANDLE_2<RESET>>;
> + };<RESET>
> +};<RESET>
> diff --git a/t/t4034/dts/post b/t/t4034/dts/post
> new file mode 100644
> index 000000000000..7803aee28093
> --- /dev/null
> +++ b/t/t4034/dts/post
> @@ -0,0 +1,32 @@
> +/ {
> + HANDLE_2: new-node@eeda {
> + compatible = "vendor,compat";
> + string-prop = end: "hello world?" start: ;
> + #address-cells = <0+40>;
> + reg = <0xeeda>;
> + prop = <(1)>;
> + prop = <(-1e10)>;
> + prop = <(!1)>;
> + prop = <(~1)>;
> + prop = <(1*2)>;
> + prop = <(1&2)>;
> + prop = <(1*2)>;
> + prop = <(1/2)>;
> + prop = <(1%2)>;
> + prop = <(1+2)>;
> + prop = <(1-2)>;
> + prop = /bits/ 32 <(1<<2)>;
> + prop = <(1>>2)>;
> + prop = <(1&2)>;
> + prop = <(1^2)>;
> + prop = <(1|2)>;
> + prop = <(1&&2)>;
> + prop = <(1||2)>;
> + prop = <(1?2:3)>;
> + list = <&HANDLE_2>, <0 0 0 1>;
> + };
> +
> + &phandle2 {
> + prop_handle = <&HANDLE_2>;
> + };
> +};
> diff --git a/t/t4034/dts/pre b/t/t4034/dts/pre
> new file mode 100644
> index 000000000000..b6a905113c22
> --- /dev/null
> +++ b/t/t4034/dts/pre
> @@ -0,0 +1,32 @@
> +/ {
> + this_handle: node@f00 {
> + compatible = "mydev";
> + string-prop = start: "hello world!" end: ;
> + #size-cells = <0+0>;
> + reg = <0xf00>;
> + prop = <1>;
> + prop = <-1e10>;
> + prop = <(!3)>;
> + prop = <(~3)>;
> + prop = <(3*4)>;
> + prop = <(3&4)>;
> + prop = <(3*4)>;
> + prop = <(3/4)>;
> + prop = <(3%4)>;
> + prop = <(3+4)>;
> + prop = <(3-4)>;
> + prop = /bits/ 64 <(3<<4)>;
> + prop = <(3>>4)>;
> + prop = <(3&4)>;
> + prop = <(3^4)>;
> + prop = <(3|4)>;
> + prop = <(3&&4)>;
> + prop = <(3||4)>;
> + prop = <(4?5:3)>;
> + list = <&this_handle>, <0 0 0 0>;
> + };
> +
> + &phandle {
> + pre-phandle = <&this_handle>;
> + };
> +};
> diff --git a/userdiff.c b/userdiff.c
> index e74a6d402255..1db5d30aaebe 100644
> --- a/userdiff.c
> +++ b/userdiff.c
> @@ -23,6 +23,15 @@ IPATTERN("ada",
> "[a-zA-Z][a-zA-Z0-9_]*"
> "|[-+]?[0-9][0-9#_.aAbBcCdDeEfF]*([eE][+-]?[0-9_]+)?"
> "|=>|\\.\\.|\\*\\*|:=|/=|>=|<=|<<|>>|<>"),
> +PATTERNS("dts",
> + /* Node name with optional label and unit address */
> + "^[ \t]*((([a-zA-Z_][a-zA-Z0-9_]*:[ \t]*)?[a-zA-Z][a-zA-Z0-9,._+-]*(@[a-zA-Z0-9,._+-]+)?"
From the examples I see in this patch, it looks like lines ending in a
';' are not candidates, everything that begins with 'word' or '&word'
is. Wouldn't that greatly simplify these patterns?
"!;\n"
/* lines beginning with a word optionally preceded by '&' */
"^[ \t]*(&?([a-zA-Z_].*)"
> + /* Reference */
> + "|&[a-zA-Z_][a-zA-Z0-9_]*)[ \t]*[^;]*)$",
Note that you don't have to replicate the syntax faithfully in the
patterns because you can assume that files adhere to the correct syntax.
You could merge this into the former pattern by just matching "&?" after
the initial whitespace.
> + /* -- */
> + /* Property names and math operators */
> + "[a-zA-Z0-9,._+?#-]+"
> + "|[-+*/%&^|!~]|>>|<<|&&|\\|\\|"),
> IPATTERN("fortran",
> "!^([C*]|[ \t]*!)\n"
> "!^[ \t]*MODULE[ \t]+PROCEDURE[ \t]\n"
>
-- Hannes
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] userdiff: Add a builtin pattern for dts files
2019-08-19 18:40 ` Johannes Sixt
@ 2019-08-19 20:44 ` Stephen Boyd
2019-08-19 21:35 ` Johannes Sixt
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Boyd @ 2019-08-19 20:44 UTC (permalink / raw)
To: Johannes Sixt
Cc: git, Adrian Johnson, William Duclot, Matthieu Moy, devicetree,
Alban Gruin, Junio C Hamano, Rob Herring, Frank Rowand
Quoting Johannes Sixt (2019-08-19 11:40:47)
> Am 17.08.19 um 00:56 schrieb Stephen Boyd:
> > The Linux kernel receives many patches to the devicetree files each
> > release. The hunk header for those patches typically show nothing,
> > making it difficult to figure out what node is being modified without
> > applying the patch or opening the file and seeking to the context. Let's
> > add a builtin 'dts' pattern to git so that users can get better diff
> > output on dts files when they use the diff=dts driver.
> >
> > The regex has been constructed based on the spec at devicetree.org[1]
> >
> > [1] https://github.com/devicetree-org/devicetree-specification/releases/latest
> >
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Signed-off-by: Stephen Boyd <sboyd@kernel.org>
> > ---
> >
> > Sending this again after getting feedback and it getting stuck in
> > review[1]. I'm not sure what happened with the meta question from Junio
> > to add a way for various projects to introduce their own patterns, but
> > I'd still prefer to have this in git proper because the kernel uses git
> > extensively and we rely on git formatted patches in our workflow. I
> > recently reviewed a dts change and remembered this never got accepted.
> >
> > Changes from v1:
> > * Updated regex to handle anything after node names instead of
> > requiring a '{'
> > * Updated test for boolean relation operators
> > * Sent out a patch to devicetree spec to document % operator
> >
> > [1] Feedback was in 16335abe-5e7e-fd7a-25f4-373f94e176e1@gmail.com
>
> Thanks. I've a few suggestions below.
>
> > diff --git a/t/t4018/dts-labels b/t/t4018/dts-labels
> > new file mode 100644
> > index 000000000000..27cd4921cfb6
> > --- /dev/null
> > +++ b/t/t4018/dts-labels
> > @@ -0,0 +1,8 @@
> > +/ {
> > + label_1: node1@ff00 {
> > + label2: RIGHT {
> > + vendor,some-property;
> > + ChangeMe = <0x45-30>;
>
> In these tests, it would be worthwhile to leave another (possibly blank)
> line before the ChangeMe line in order to demonstrate that lines
> beginning with a word, such as the 'vendor,some-property;' line, are
> _not_ picked up when they are not in the hunk context.
Sure. I can add a blank line. Did you want it on all the tests or just
some of them?
> > diff --git a/userdiff.c b/userdiff.c
> > index e74a6d402255..1db5d30aaebe 100644
> > --- a/userdiff.c
> > +++ b/userdiff.c
> > @@ -23,6 +23,15 @@ IPATTERN("ada",
> > "[a-zA-Z][a-zA-Z0-9_]*"
> > "|[-+]?[0-9][0-9#_.aAbBcCdDeEfF]*([eE][+-]?[0-9_]+)?"
> > "|=>|\\.\\.|\\*\\*|:=|/=|>=|<=|<<|>>|<>"),
> > +PATTERNS("dts",
> > + /* Node name with optional label and unit address */
> > + "^[ \t]*((([a-zA-Z_][a-zA-Z0-9_]*:[ \t]*)?[a-zA-Z][a-zA-Z0-9,._+-]*(@[a-zA-Z0-9,._+-]+)?"
>
> From the examples I see in this patch, it looks like lines ending in a
> ';' are not candidates, everything that begins with 'word' or '&word'
> is. Wouldn't that greatly simplify these patterns?
>
> "!;\n"
> /* lines beginning with a word optionally preceded by '&' */
> "^[ \t]*(&?([a-zA-Z_].*)"
Right. I was stuck with my old regex ways where it wasn't considering
lines that didn't end in a semicolon. This looks like it will work.
>
> > + /* Reference */
> > + "|&[a-zA-Z_][a-zA-Z0-9_]*)[ \t]*[^;]*)$",
>
> Note that you don't have to replicate the syntax faithfully in the
> patterns because you can assume that files adhere to the correct syntax.
> You could merge this into the former pattern by just matching "&?" after
> the initial whitespace.
Ok. Thanks for simplifying by providing the regex above. I'll rework and
resend.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] userdiff: Add a builtin pattern for dts files
2019-08-19 20:44 ` Stephen Boyd
@ 2019-08-19 21:35 ` Johannes Sixt
0 siblings, 0 replies; 6+ messages in thread
From: Johannes Sixt @ 2019-08-19 21:35 UTC (permalink / raw)
To: Stephen Boyd
Cc: git, Adrian Johnson, William Duclot, Matthieu Moy, devicetree,
Alban Gruin, Junio C Hamano, Rob Herring, Frank Rowand
Am 19.08.19 um 22:44 schrieb Stephen Boyd:
> Quoting Johannes Sixt (2019-08-19 11:40:47)
>> Am 17.08.19 um 00:56 schrieb Stephen Boyd:
>>> diff --git a/t/t4018/dts-labels b/t/t4018/dts-labels
>>> new file mode 100644
>>> index 000000000000..27cd4921cfb6
>>> --- /dev/null
>>> +++ b/t/t4018/dts-labels
>>> @@ -0,0 +1,8 @@
>>> +/ {
>>> + label_1: node1@ff00 {
>>> + label2: RIGHT {
>>> + vendor,some-property;
>>> + ChangeMe = <0x45-30>;
>>
>> In these tests, it would be worthwhile to leave another (possibly blank)
>> line before the ChangeMe line in order to demonstrate that lines
>> beginning with a word, such as the 'vendor,some-property;' line, are
>> _not_ picked up when they are not in the hunk context.
>
> Sure. I can add a blank line. Did you want it on all the tests or just
> some of them?
Some of them. We should have cases that show that
"vendor,some-property;" and "prop-erty = value;" are not picked up.
-- Hannes
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-08-19 21:35 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-16 22:56 [PATCH v2] userdiff: Add a builtin pattern for dts files Stephen Boyd
2019-08-17 1:48 ` Frank Rowand
2019-08-17 15:18 ` Alban Gruin
2019-08-19 18:40 ` Johannes Sixt
2019-08-19 20:44 ` Stephen Boyd
2019-08-19 21:35 ` Johannes Sixt
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).