Skip to content

Conversation

@jrklasen
Copy link
Contributor

@jrklasen jrklasen commented Dec 8, 2025

This PR adds a build recipe for OSRM for all Linux, Windows, and Apple platforms.

The goal is a Julia wrapper around OSRM-backend. This has three major steps: i. building the OSRM backend with BinaryBuilder.jl (this PR); ii. building a C API for libosrm and the corresponding build recipe (which you can find in this draft PR #12744); iii. a Julia wrapper, which can be found in this repository. There you can see the final result; it works already with some local builds of OSRM_jll and libosrmc_jll

I would appreciate it if @jeremiahpslewis and @mattwigway could take a look at this.

@fingolfin I saw your command at #8893 about the macOS SDK; unfortunately, I could not get this working here

Pair programmed with an AI agent

Closes: #8893

@mattwigway
Copy link
Contributor

Thanks, I'll take a look over the break!

@fingolfin
Copy link
Member

@fingolfin I saw your command at #8893 about the macOS SDK; unfortunately, I could not get this working here

What exactly was the problem?

@jrklasen
Copy link
Contributor Author

jrklasen commented Dec 9, 2025

The way get_macos_sdk_script works resulted in some permission issues I was unable to track down.

@fingolfin
Copy link
Member

Normally you wouldn't use get_macos_sdk_script, though, but rather

sources, script = require_macos_sdk("14.5", sources, script)

@jrklasen
Copy link
Contributor Author

Sorry, for not explaining the problem well.

Your recommendation was implemented; however, subsequent execution of the script resulted in permission errors. It works, though, when I extract the SDK the way it is done in this PR.

I was thinking the discrepancy between the extraction in this PR and "get_macos_sdk_script" was the root cause; however, I was unable to determine the precise issue.

Is a manual extraction of the SDK a blocker for merging this, or can we leave it like this for now?

@fingolfin fingolfin merged commit b562271 into JuliaPackaging:master Dec 10, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants