Opened 17 months ago

Closed 16 months ago

Last modified 16 months ago

#67566 closed defect (fixed)

py*-grpcio @1.48.0+_7 does not build

Reported by: josephsacco Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: Cc: brolcz, someuser12, jbrundle, emcrisostomo (Enrico Maria Crisostomo), barracuda156
Port: grpc

Description

py38-grpcio, py39-grpcio, py310-grpcio all fail to build:

Exit code: 1
Error: Failed to configure py310-grpcio: configure failure: command execution failed

Log file from py38-grpcio attached.

Attachments (1)

main.log (49.8 KB) - added by josephsacco 17 months ago.
Log file for py38-grpcio

Download all attachments as: .zip

Change History (20)

Changed 17 months ago by josephsacco

Attachment: main.log added

Log file for py38-grpcio

comment:1 Changed 17 months ago by ryandesign (Ryan Carsten Schmidt)

Cc: mascguy added
Keywords: python removed
Owner: set to emcrisostomo
Port: grpc added; py-grpcio removed
Status: newassigned

Relevant errors in the log start with:

CMake Error at /opt/local/lib/cmake/absl/abslTargets.cmake:42 (message):

I would say this has something to do with the recent updates of protobuf3-cpp.

comment:2 Changed 17 months ago by mascguy (Christopher Nielsen)

Cc: barracuda156 added

This is actually a combination of things - starting with the recent update to re2, which added abseil - plus other things too.

Regardless, our grpcio port is outdated, and Sergey is in the the process of creating a PR to update it. The only roadblock was adding cmake files for protobuf3-cpp, which is now done.

comment:3 Changed 17 months ago by ryandesign (Ryan Carsten Schmidt)

Cc: brolcz added

Has duplicate #67662.

comment:4 Changed 17 months ago by mascguy (Christopher Nielsen)

Sergey, are you still working on creating a PR to fix py-grpcio?

comment:5 Changed 17 months ago by someuser12

Cc: someuser12 added

comment:6 Changed 16 months ago by ryandesign (Ryan Carsten Schmidt)

Cc: jbrundle added

Has duplicate #67721.

comment:7 in reply to:  4 ; Changed 16 months ago by barracuda156

Replying to mascguy:

Sergey, are you still working on creating a PR to fix py-grpcio?

Sorry, will return to that soon. I need the PR for arrow merged :)

comment:8 in reply to:  7 Changed 16 months ago by mascguy (Christopher Nielsen)

Replying to barracuda156:

Replying to mascguy:

Sergey, are you still working on creating a PR to fix py-grpcioh?

Sorry, will return to that soon. I need the PR for arrow merged :)

Merged, so go ahead and submit a PR for your update to py-grpcio.

comment:9 Changed 16 months ago by jbrundle

Waiting eagerly for this to be fixed!

comment:10 Changed 16 months ago by mascguy (Christopher Nielsen)

Cc: emcrisostomo added; barracuda156 removed
Owner: changed from emcrisostomo to barracuda156

Sergey, are you working on this?

comment:11 Changed 16 months ago by mascguy (Christopher Nielsen)

Cc: barracuda156 added; mascguy removed
Owner: changed from barracuda156 to mascguy

comment:12 Changed 16 months ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In e8f2531d7f885d04c618433111863bf039587431/macports-ports (master):

py-grpcio*: don't build/use internal abseil
Fixes: #67566

comment:13 Changed 16 months ago by jbrundle

Thanks for fixing this. I note that with my M2 silicon, py38-grpcio builds, but so far py310-grpcio apparently does not. Perhaps this is OK?

comment:14 Changed 16 months ago by josephsacco

Hmmm... On an intel-based iMac running MacOS 12.6.7 all is now well:

port installed | grep grpcio
  py38-grpcio @1.48.0_8 (active)
  py39-grpcio @1.48.0_8 (active)
  py310-grpcio @1.48.0_8 (active)

Is there more to be done?

-Joseph

comment:15 Changed 16 months ago by jbrundle

Doing further tests on my Mac mini M2 chip now. Should.d have results imminently

comment:16 Changed 16 months ago by jbrundle

py310-grpcio does build on my MacBook Pro running Monterey 12.6.7 with Intel i5 silicon

comment:17 Changed 16 months ago by jbrundle

So now I find that py310-grpcio *does* build on my M2 silicon running Ventura 13.4.1. So please ignore my previous comment

comment:18 Changed 16 months ago by jbrundle

Many thanks for your quick work on this problem. I will be using this package for building science transformers

comment:19 in reply to:  18 Changed 16 months ago by mascguy (Christopher Nielsen)

Replying to jbrundle:

Many thanks for your quick work on this problem. I will be using this package for building science transformers

You're very welcome, glad the issue is resolved!

Note: See TracTickets for help on using tickets.