Opened 4 years ago

Last modified 4 years ago

#52092 assigned enhancement

mp-buildbot: use tools curl instead of macOS curl

Reported by: ryandesign (Ryan Schmidt) Owned by: larryv (Lawrence Velázquez)
Priority: Normal Milestone:
Component: contrib Version:
Keywords: mpbb Cc: neverpanic (Clemens Lang), raimue (Rainer Müller), mojca (Mojca Miklavec)
Port:

Description

mpbb-gather-archives currently hardcodes:

    curl=/usr/bin/curl

I'd like this to use the curl installed in the tools prefix instead. /usr/bin/curl on Leopard can't understand our SSL certificate.

The way that mpbb-checkout uses the tools svn is via a --svn flag. To use the tools curl, do we need to add a --curl flag?

Change History (6)

comment:1 in reply to:  description ; Changed 4 years ago by larryv (Lawrence Velázquez)

Keywords: mpbb added
Owner: changed from macports-tickets@… to larryv@…
Status: newassigned

Replying to ryandesign@…:

I'd like this to use the curl installed in the tools prefix instead. /usr/bin/curl on Leopard can't understand our SSL certificate.

I’m rewriting mpbb at the moment. I’ll add this functionality.

The way that mpbb-checkout uses the tools svn is via a --svn flag. To use the tools curl, do we need to add a --curl flag?

Not necessarily, but it wouldn’t be hard to provide.

comment:2 in reply to:  1 Changed 4 years ago by ryandesign (Ryan Schmidt)

Replying to larryv@…:

I’m rewriting mpbb at the moment.

That's exciting! Good luck.

I’ll add this functionality.

Thanks!

comment:3 Changed 4 years ago by ryandesign (Ryan Schmidt)

Just checking if any progress has been made on this.

comment:4 in reply to:  3 Changed 4 years ago by larryv (Lawrence Velázquez)

I just removed the hardcoded path so that the curl executable is found through PATH. Is that sufficient to get the Leopard buildslaves working? (I’ll add the command-line option later.)

comment:5 Changed 4 years ago by ryandesign (Ryan Schmidt)

Yes, build 696 was able to determine which archives had previously been uploaded and only staged the new ones, whereas build 695 was still staging all of them. Thanks.

comment:6 in reply to:  5 Changed 4 years ago by larryv (Lawrence Velázquez)

Okay, great. I’ll leave this ticket open to remind myself to add that command-line option.

Note: See TracTickets for help on using tickets.