squid: build failure for Impish RISC-V

Bug #1934891 reported by Heinrich Schuchardt
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
squid (Ubuntu)
Fix Released
High
Thomas Ward

Bug Description

Building Squid for Impish on RISC-V fails.

Two problems were identified. Upstream merge request have been created:

https://github.com/squid-cache/squid/pull/855 for

tools.cc: In function ‘void enter_suid()’:
tools.cc:585:11: error: ignoring return value of ‘int setuid(__uid_t)’
declared with attribute ‘warn_unused_result’ [-Werror=unused-result]
  585 | setuid(0);
      | ~~~~~~^~~

https://github.com/squid-cache/squid/pull/856 for

/usr/riscv64-linux-gnu/include/c++/10/ostream:611:
undefined reference to __atomic_compare_exchange_1

With suggested patches building succeeds.

Best regards

Heinrich

Tags: patch

Related branches

summary: - squid: build failure for Impish RISCV
+ squid: build failure for Impish RISC-V
Changed in squid (Ubuntu):
assignee: nobody → Heinrich Schuchardt (xypron)
status: New → In Progress
Revision history for this message
Heinrich Schuchardt (xypron) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Patch" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
Heinrich Schuchardt (xypron) wrote :

Upstream patches have been merged in https://github.com/squid-cache/squid.

Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Hi Heinrich,

Since you've assigned this to yourself and set the status to "In Progress" and attached a debdiff, I am hoping that you're taking care of this and that this would need no help from the Server team side?

Revision history for this message
Heinrich Schuchardt (xypron) wrote :

A sponsor taking care of pushing the changes is needed as I have no upload rights.

Revision history for this message
Thomas Ward (teward) wrote :

I'll handle the updating and pushing to Impish later today after my errands, which will be when I'm in front of a computer.

Changed in squid (Ubuntu):
status: In Progress → Confirmed
assignee: Heinrich Schuchardt (xypron) → nobody
assignee: nobody → Thomas Ward (teward)
status: Confirmed → Triaged
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Thanks, Thomas. I was gonna look at it on Monday but thanks for beating me to it! :D

Thomas Ward (teward)
Changed in squid (Ubuntu):
status: Triaged → In Progress
importance: Undecided → High
Revision history for this message
Thomas Ward (teward) wrote :

Uploaded to impish-proposed. Should be building, and provided no autopkgtests fail it should migrate to impish repos on its own.

Changed in squid (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Thomas Ward (teward) wrote :

Unsubscribing sponsors as I've sponsored this. Server Team: do me a solid and keep an eye out for any autopkgtest failures in case I miss them - I have a lot of mail so sometimes I don't see the failure notices :P

Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

> Server Team: do me a solid and keep an eye out for any
> autopkgtest failures in case I miss them - I have a lot
> of mail so sometimes I don't see the failure notices :P

Aye, aye, captain! Will do!

Revision history for this message
Heinrich Schuchardt (xypron) wrote :
Revision history for this message
Bryce Harrington (bryce) wrote :

Autopkgtests passed as well. All looks good to go.
squid is blocked due to openldap's transition (which itself is blocked by gnome-shell IIRC).
But looks like squid will go through without issue once all that has cleared.

https://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#squid

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package squid - 4.13-10ubuntu3

---------------
squid (4.13-10ubuntu3) impish; urgency=medium

  * Fix failure to build on RISC-V (LP: #1934891)

 -- Heinrich Schuchardt <email address hidden> Wed, 07 Jul 2021 14:11:51 +0200

Changed in squid (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Patches

Remote bug watches

Bug watches keep track of this bug in other bug trackers.