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

gjs: update to 1.54.3 #12192

Merged
merged 1 commit into from
Sep 19, 2021
Merged

gjs: update to 1.54.3 #12192

merged 1 commit into from
Sep 19, 2021

Conversation

evanmiller
Copy link
Contributor

@evanmiller evanmiller commented Sep 10, 2021

Description

This update:

  • Upgrades gjs version from 1.52.5 to 1.54.3
  • Bumps the mozjs dependency to mozjs60
  • Removes the obsolete compiler blacklist

Originally this PR included PowerPC support requiring #12186, but that change is still WIP and may not require alterations to this Portfile so I will keep it separate.

Debian's 1.52 -> 1.54 migration notes can be found here:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=906016

The conclusion of that discussion was that the included mozjs52 -> mozjs60 transition does not require a soname bump, so the reverse dependents should not require a rebuild.

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.4.11 8S165 Power Macintosh
Component versions: DevToolsCore-798.0; DevToolsSupport-794.0

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@macportsbot
Copy link

Notifying maintainers:
@dbevans for port gjs.

@evanmiller
Copy link
Contributor Author

After some further experimenting I realized that the PPC support can be achieved later without #12186 being merged first – so this version bump is ready for review. (Updated description above.)

@evanmiller evanmiller marked this pull request as ready for review September 10, 2021 23:29
@evanmiller
Copy link
Contributor Author

Updated to remove the earlier 32-bit patch since adding it later will not necessitate a revbump.

@herbygillot herbygillot merged commit cb7bfd7 into macports:master Sep 19, 2021
@evanmiller evanmiller mentioned this pull request Sep 19, 2021
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants