Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use maplibre instead of mapbox style spec #940

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ahocevar
Copy link
Member

I'm putting this up for discussion, as an alternative to #939.

The one thing that makes me hesitant to make this change is that I don't know what the plans are for Maplibre to stick with or depart from the Mapbox style spec. If someone from @maplibre, maybe @birkskyum, could comment on that, it would make the decision easier.

@ahocevar ahocevar marked this pull request as draft June 30, 2023 12:46
@birkskyum
Copy link

birkskyum commented Jun 30, 2023

Sure, since the Mapbox style spec has become proprietary, the plan for MapLibre is to continue developing the MapLibre Style Spec independently from Mapbox. The main divergence has so far been around atmosphere / globe features which were added after the fork, but the two style specs are still largely identical, and most changes happen as backwards-compatible additions, with both style specs still being on version 8.x.

@mbeckem
Copy link

mbeckem commented Sep 15, 2023

@ahocevar It appears that new versions of @mapbox/mapbox-gl-style-spec will also fall under the proprietary license as of this commit: mapbox/mapbox-gl-js@a02c71b. So switching to MapLibre Style Spec seems like a good idea moving forward.

@ahocevar
Copy link
Member Author

ahocevar commented Sep 15, 2023

@mbeckem Thanks for the pointer!

If you are interested in using this module in a manner not expressly permitted by its license, please do not hesitate to contact legal@mapbox.com with details of what you have in mind.

I'll do this first, and see what the options are.

@orangemug
Copy link
Contributor

I'm in favour of moving over to @maplibre/maplibre-gl-style-spec, reasons being:

  • we have an opportunity to discuss any changes with the maplibre team (because it's public)
  • the maplibre spec is open source, no issues with licensing
  • I'm personally building a tool on-top of maplibre/openlayer 😄

@ahocevar
Copy link
Member Author

Since I got no satisfying answer from Mapbox, I'm inclined to go in that direction too. But it's also tempting to create a new package, independent of both Mapbox and Maplibre, with a few more useful features (profiles, auto-generated docs, optimized for build size).

@orangemug
Copy link
Contributor

@ahocevar the spec is in progress over at https://github.com/maparatus/ol-mapbox-style-spec, rendered spec at https://maparatus.github.io/ol-mapbox-style-spec/ (I'll continue to update issue #10 on progress).

That's based on the maplibre-gl spec. I'm happy to transfer that repository over once I complete the tests and update the spec.

From my perspective. It'd be fine to merge this in the meantime, as it'd be closer to the incoming spec work.

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.

None yet

4 participants