mirrors/arti.git/maint/check_licenses, branch tor-llcrypto-v0.4.4mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=tor-llcrypto-v0.4.42023-02-24T17:27:43Zmaint/check_licensse: Placate shellcheck2023-02-24T17:27:43ZIan Jacksonijackson@chiark.greenend.org.uk2023-02-24T17:27:43Zurn:sha1:df79d407d93a97493e4999a274fa79997dce827a
There's nothing wrong with the previous idiom since it's a trusted
variable, but whatever.
maint/check_licensse: Approve Zlib licence2023-02-24T17:14:27ZIan Jacksonijackson@chiark.greenend.org.uk2023-02-24T17:13:24Zurn:sha1:4756ad6e5733adee9da864d3195f18155ade4390
This is a very permissive BSD/MIT-like licence. I have reviewed the
licence text and it is inoffensive.
I want this for slotmap, in !1034
maint/check_licensse: Honour CARGO variable2023-02-24T17:14:27ZIan Jacksonijackson@chiark.greenend.org.uk2023-02-24T17:12:14Zurn:sha1:7068f237f76e0ac40e8f5249c672934f61e89a30
Even if it contains spaces.
maint/check_licensse: Do not discard actual error2023-02-24T17:14:27ZIan Jacksonijackson@chiark.greenend.org.uk2023-02-24T17:07:37Zurn:sha1:745336fac51939917ae7a063a0470763a1e936b4
This confused me while I was debugging the interaction with nailing-cargo.
shell: use /usr/bin/env instead of absolute paths2023-01-25T17:35:53ZEmil Englerme@emilengler.com2023-01-25T17:35:53Zurn:sha1:79097484de7401bf247e14c1bdf5982aee4518a1
This commit changes the shebang in all shell scripts from absolute
paths (such as `/bin/bash` or `/usr/bin/python3`) to the `/usr/bin/env`
binary with the accompanying interpreter as it's argument.
The reason for this are as follows:
- NixOS cannot work with absolute paths
- BSD systems install their packages in /usr/local/bin
check_licenses: List license for tinystr2022-08-31T00:09:19ZNick Mathewsonnickm@torproject.org2022-08-30T21:54:19Zurn:sha1:167408050f50adef4c29850ad14959c9366dc0c5fix error running check_licenses2022-08-13T07:02:17Ztrinity-1686atrinity@deuxfleurs.fr2022-08-13T07:02:17Zurn:sha1:2339f637c5840381e69bc43821bd4687656c29cb
--version was removed in a recent update
we use that to detected whether cargo-license is installed,
now use --help instead
Several typo fixes from `typos`.2022-07-27T14:12:32ZNick Mathewsonnickm@torproject.org2022-07-27T14:12:32Zurn:sha1:dfdcc97858bd6e824c5e6aa4604746ede551780cTeach check_licenses to accept license on `unicode-ident`.2022-07-20T13:40:12ZNick Mathewsonnickm@torproject.org2022-07-20T13:39:21Zurn:sha1:275c98940a6266948487bbd4ed9a181036f0252a
The license there is `(MIT OR Apache-2.0) AND Unicode-DFS-2016`,
which fine, but the existing shell script doesn't actually handle
"AND" correctly. This commit adds a workaround for licenses that
are "AND", and some comments about weaknesses in our (lack of)
boolean expression parsing.
This should fix CI.
check_licenses: list new crates with manual license verification2022-05-31T17:58:40ZNick Mathewsonnickm@torproject.org2022-05-31T17:20:20Zurn:sha1:842e6a697d990556d39eabdf7daabed7e19f2e36
Closes #462.
Note that the license on `ring` is slightly problematic for some
users, including as it does old the openssl license[^1], with
advertising clause and all. That's not a blocker for us now, since
`ring` is not a required dependency. But we wouldn't want `ring` to
become a mandatory dependency because of this.
[^1]: To make the situation even more complicated, modern openssl
has relicensed under apache-2.0, but that doesn't necessarily
help us, since ring took its code from boringssll, which
forked from an older version of openssl.