git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* [PATCH] ci: add Cirrus-CI config for FreeBSD CI
@ 2019-11-25 20:37 Ed Maste
  2019-11-26  0:44 ` brian m. carlson
  2019-12-20  2:11 ` [PATCH v2] CI: add FreeBSD CI support via Cirrus-CI Ed Maste
  0 siblings, 2 replies; 6+ messages in thread
From: Ed Maste @ 2019-11-25 20:37 UTC (permalink / raw)
  To: git; +Cc: Ed Maste

From: Ed Maste <emaste@freebsd.org>

Cirrus-CI is relatively unique among hosted CI providers in supporting
FreeBSD (in addition to Linux, Windows, and macOS).  Add a Cirrus-CI
config to facilitate building and testing on FreeBSD.

Signed-off-by: Ed Maste <emaste@freebsd.org>
---
6 i18n tests are currently failing and need investigation.
Example result: https://cirrus-ci.com/task/5394512637067264
Test log: https://api.cirrus-ci.com/v1/task/5394512637067264/logs/test.log

 .cirrus.yml | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 .cirrus.yml

diff --git a/.cirrus.yml b/.cirrus.yml
new file mode 100644
index 0000000000..47d871ded5
--- /dev/null
+++ b/.cirrus.yml
@@ -0,0 +1,12 @@
+env:
+  CIRRUS_CLONE_DEPTH: 1
+
+freebsd_12_task:
+  freebsd_instance:
+    image: freebsd-12-1-release-amd64
+  install_script:
+    pkg install -y gettext gmake perl5
+  build_script:
+    - gmake
+  test_script:
+    - gmake test
-- 
2.24.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ci: add Cirrus-CI config for FreeBSD CI
  2019-11-25 20:37 [PATCH] ci: add Cirrus-CI config for FreeBSD CI Ed Maste
@ 2019-11-26  0:44 ` brian m. carlson
  2019-11-26 13:31   ` Ed Maste
  2019-11-27 14:01   ` Ed Maste
  2019-12-20  2:11 ` [PATCH v2] CI: add FreeBSD CI support via Cirrus-CI Ed Maste
  1 sibling, 2 replies; 6+ messages in thread
From: brian m. carlson @ 2019-11-26  0:44 UTC (permalink / raw)
  To: Ed Maste; +Cc: git, Ed Maste

[-- Attachment #1: Type: text/plain, Size: 947 bytes --]

On 2019-11-25 at 20:37:40, Ed Maste wrote:
> From: Ed Maste <emaste@freebsd.org>
> 
> Cirrus-CI is relatively unique among hosted CI providers in supporting
> FreeBSD (in addition to Linux, Windows, and macOS).  Add a Cirrus-CI
> config to facilitate building and testing on FreeBSD.
> 
> Signed-off-by: Ed Maste <emaste@freebsd.org>

I'm all for automated testing on FreeBSD, but we would need someone to
triage and address any failures reasonably quickly.  Is that something
you'd be okay with doing, or is there someone else who would be okay
with doing that?

> ---
> 6 i18n tests are currently failing and need investigation.
> Example result: https://cirrus-ci.com/task/5394512637067264
> Test log: https://api.cirrus-ci.com/v1/task/5394512637067264/logs/test.log

Could we fix these issues first so we don't have CI suddenly start
failing?
-- 
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 868 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ci: add Cirrus-CI config for FreeBSD CI
  2019-11-26  0:44 ` brian m. carlson
@ 2019-11-26 13:31   ` Ed Maste
  2019-11-27 14:01   ` Ed Maste
  1 sibling, 0 replies; 6+ messages in thread
From: Ed Maste @ 2019-11-26 13:31 UTC (permalink / raw)
  To: brian m. carlson, git; +Cc: avarab

On Mon, 25 Nov 2019 at 19:44, brian m. carlson
<sandals@crustytoothpaste.net> wrote:
>
> On 2019-11-25 at 20:37:40, Ed Maste wrote:
> > From: Ed Maste <emaste@freebsd.org>
> >
> > Cirrus-CI is relatively unique among hosted CI providers in supporting
> > FreeBSD (in addition to Linux, Windows, and macOS).  Add a Cirrus-CI
> > config to facilitate building and testing on FreeBSD.
> >
> > Signed-off-by: Ed Maste <emaste@freebsd.org>
>
> I'm all for automated testing on FreeBSD, but we would need someone to
> triage and address any failures reasonably quickly.  Is that something
> you'd be okay with doing, or is there someone else who would be okay
> with doing that?

We're currently experimenting with a migration of the FreeBSD repo
from Subversion to git, so long term there will be many with a vested
interest in triaging and addressing failures. In the near term though
I'd be able to take this on.

> > 6 i18n tests are currently failing and need investigation.
> > Example result: https://cirrus-ci.com/task/5394512637067264
> > Test log: https://api.cirrus-ci.com/v1/task/5394512637067264/logs/test.log
>
> Could we fix these issues first so we don't have CI suddenly start
> failing?

Indeed, that makes sense. I think the failures may be an issue with
the test though; here's the first failure:

expecting success of 4210.6 '-c grep.patternType=fixed log --grep does not find
non-reencoded values (latin1 + locale)':
                cat >expect <<-\EOF &&
                latin1
                utf8
                EOF
                LC_ALL="is_IS.UTF-8" git -c grep.patternType=fixed log
--encoding=ISO-8859-1 --format=%s --grep="_" >actual &&
                test_cmp expect actual

fatal: command line, '_': illegal byte sequence

This was added in 4e2443b1813 with this note in the commit message:

| It's possible that this
| test breaks the "basic" and "extended" backends on some systems that
| are more anal than glibc about the encoding of locale issues with
| POSIX functions that I can remember, but PCRE is more careful about
| the validation.

I've CC'd Ævar Arnfjörð Bjarmason for advice on this.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ci: add Cirrus-CI config for FreeBSD CI
  2019-11-26  0:44 ` brian m. carlson
  2019-11-26 13:31   ` Ed Maste
@ 2019-11-27 14:01   ` Ed Maste
  2019-11-27 15:17     ` Ed Maste
  1 sibling, 1 reply; 6+ messages in thread
From: Ed Maste @ 2019-11-27 14:01 UTC (permalink / raw)
  To: brian m. carlson, git

On Mon, 25 Nov 2019 at 19:44, brian m. carlson
<sandals@crustytoothpaste.net> wrote:
>
> I'm all for automated testing on FreeBSD, but we would need someone to
> triage and address any failures reasonably quickly.  Is that something
> you'd be okay with doing, or is there someone else who would be okay
> with doing that?

Renato Botelho <garga@freebsd.org> is the maintainer of the devel/git
port/package on FreeBSD and has now also joined this list and offered
to help triage and address issues.

> Could we fix these issues first so we don't have CI suddenly start
> failing?

After some further investigation I believe these tests are technically
invalid but work with glibc because of its looser requirements. I've
sent a patch (t4210: skip i18n tests that don't work on FreeBSD) to
address the failures.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ci: add Cirrus-CI config for FreeBSD CI
  2019-11-27 14:01   ` Ed Maste
@ 2019-11-27 15:17     ` Ed Maste
  0 siblings, 0 replies; 6+ messages in thread
From: Ed Maste @ 2019-11-27 15:17 UTC (permalink / raw)
  To: brian m. carlson, git

On Wed, 27 Nov 2019 at 09:01, Ed Maste <emaste@freebsd.org> wrote:
>
> After some further investigation I believe these tests are technically
> invalid but work with glibc because of its looser requirements. I've
> sent a patch (t4210: skip i18n tests that don't work on FreeBSD) to
> address the failures.

And a now-passing test run on FreeBSD with the change :
https://cirrus-ci.com/task/5657957240406016

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] CI: add FreeBSD CI support via Cirrus-CI
  2019-11-25 20:37 [PATCH] ci: add Cirrus-CI config for FreeBSD CI Ed Maste
  2019-11-26  0:44 ` brian m. carlson
@ 2019-12-20  2:11 ` Ed Maste
  1 sibling, 0 replies; 6+ messages in thread
From: Ed Maste @ 2019-12-20  2:11 UTC (permalink / raw)
  To: git mailing list; +Cc: brian m . carlson, Ed Maste

Currently testing on FreeBSD 12.1.

Signed-off-by: Ed Maste <emaste@FreeBSD.org>
---
Since v1, create an unprivileged user and build/test using that user.
CI run with this change: https://cirrus-ci.com/task/6565294088126464

 .cirrus.yml | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
 create mode 100644 .cirrus.yml

diff --git a/.cirrus.yml b/.cirrus.yml
new file mode 100644
index 0000000000..c2f5fe385a
--- /dev/null
+++ b/.cirrus.yml
@@ -0,0 +1,15 @@
+env:
+  CIRRUS_CLONE_DEPTH: 1
+
+freebsd_12_task:
+  freebsd_instance:
+    image: freebsd-12-1-release-amd64
+  install_script:
+    pkg install -y gettext gmake perl5
+  create_user_script:
+    - pw useradd git
+    - chown -R git:git .
+  build_script:
+    - su git -c gmake
+  test_script:
+    - su git -c 'gmake test'
-- 
2.24.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-12-20  2:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-25 20:37 [PATCH] ci: add Cirrus-CI config for FreeBSD CI Ed Maste
2019-11-26  0:44 ` brian m. carlson
2019-11-26 13:31   ` Ed Maste
2019-11-27 14:01   ` Ed Maste
2019-11-27 15:17     ` Ed Maste
2019-12-20  2:11 ` [PATCH v2] CI: add FreeBSD CI support via Cirrus-CI Ed Maste

Code repositories for project(s) associated with this 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).