[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#988228: marked as done (unblock: golang-testify/1.6.1-2)



Your message dated Sat, 08 May 2021 19:20:30 +0000
with message-id <E1lfSV4-0007PY-SG@respighi.debian.org>
and subject line unblock golang-testify
has caused the Debian Bug report #988228,
regarding unblock: golang-testify/1.6.1-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
988228: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=988228
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc: nilesh@debian.org

Please unblock package golang-testify

[ Reason ]
The build fails on armhf architecture, due to a failing build time test.
This has been fixed in 1.6.1-2

It also fixes the corresponding RC bug: #988101

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=988101

[ Impact ]
The package will fail to build from source on armhf

[ Tests ]
It has non-trivial autopkgtests that cover the testing to the changes.
Autopkgtests pass for golang-testify
It also looks good when I tested it locally

[ Risks ]
No risks, trivial change to make it buildable across armhf.

[ Checklist ]  
  [x] all changes are documented in the d/changelog
  [x] I reviewed all changes and I approve them
  [x] attach debdiff against the package in testing

unblock golang-testify/1.6.1-2

-- System Information:
Debian Release: bullseye/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 5.7.0-2-amd64 (SMP w/8 CPU threads)
Kernel taint flags: TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
diff -Nru golang-testify-1.6.1/debian/changelog golang-testify-1.6.1/debian/changelog
--- golang-testify-1.6.1/debian/changelog	2020-09-28 04:00:48.000000000 +0000
+++ golang-testify-1.6.1/debian/changelog	2021-05-08 04:25:09.000000000 +0000
@@ -1,3 +1,11 @@
+golang-testify (1.6.1-2) unstable; urgency=medium
+
+  * Team Upload.
+  * d/p/do-not-use-race.patch: Drop -race flag with
+    golang in TestFailfastSuiteFailFastOn (Closes: #988101)
+
+ -- Nilesh Patra <nilesh@debian.org>  Sat, 08 May 2021 12:25:09 +0800
+
 golang-testify (1.6.1-1) unstable; urgency=medium
 
   * Team upload.
diff -Nru golang-testify-1.6.1/debian/patches/do-not-use-race.patch golang-testify-1.6.1/debian/patches/do-not-use-race.patch
--- golang-testify-1.6.1/debian/patches/do-not-use-race.patch	1970-01-01 00:00:00.000000000 +0000
+++ golang-testify-1.6.1/debian/patches/do-not-use-race.patch	2021-05-08 04:25:09.000000000 +0000
@@ -0,0 +1,19 @@
+Description: The "-race" flag for golang is available only for amd64, arm64, ppc64el. Since this is being used only in a test, it can be dropped
+Author: Nilesh Patra <nilesh@debian.org>
+Bug-Debian: https://bugs.debian.org/988101
+Last-Update: 2021-05-08
+--- a/suite/suite_test.go
++++ b/suite/suite_test.go
+@@ -552,10 +552,10 @@
+ }
+ func TestFailfastSuiteFailFastOn(t *testing.T) {
+ 	// To test this with failfast on (and isolated from other intended test failures in our test suite) we launch it in its own process
+-	cmd := exec.Command("go", "test", "-v", "-race", "-run", "TestFailfastSuite", "-failfast")
++	cmd := exec.Command("go", "test", "-v", "-run", "TestFailfastSuite", "-failfast")
+ 	var out bytes.Buffer
+ 	cmd.Stdout = &out
+-	t.Log("Running go test -v -race -run TestFailfastSuite -failfast")
++	t.Log("Running go test -v -run TestFailfastSuite -failfast")
+ 	err := cmd.Run()
+ 	t.Log(out.String())
+ 	if err != nil {
diff -Nru golang-testify-1.6.1/debian/patches/series golang-testify-1.6.1/debian/patches/series
--- golang-testify-1.6.1/debian/patches/series	1970-01-01 00:00:00.000000000 +0000
+++ golang-testify-1.6.1/debian/patches/series	2021-05-08 04:25:09.000000000 +0000
@@ -0,0 +1 @@
+do-not-use-race.patch

--- End Message ---
--- Begin Message ---
Unblocked.

--- End Message ---

Reply to: