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

GtkWave : Update gtkwave to 3.3.117 #22128

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

langly
Copy link
Contributor

@langly langly commented Jan 11, 2024

Description

Updated GtkWave from 3.3.115 to 3.3.117.

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

macOS 14.0 23A344 arm64
Xcode 15.2 15C500b

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 --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@pmetzger
Copy link
Member

CI failed.

@langly
Copy link
Contributor Author

langly commented Jan 11, 2024

I saw. Seems like it's falling over on one of the dependencies. I'll check later.

@langly
Copy link
Contributor Author

langly commented Jan 12, 2024

Ah, yeah. Found some time to look at it now. The problem is not in this Portfile, for the macos-11 build, but in gtk-osx-application-common-gtk3.

From the logfiles:

DEBUG: Using compiler 'Xcode Clang'
DEBUG: Executing proc-pre-org.macports.configure-configure-0
DEBUG: Active variants check for source-type install considers depends_fetch depends_extract depends_lib depends_build depends_run: xz gtk3 pkgconfig
DEBUG: Executing proc-pre-org.macports.configure-configure-1
DEBUG: gtk3 is installed with the following variants: +x11
DEBUG:   required: quartz, forbidden:
DEBUG:   rejected, because required variant quartz is missing
Error:
gtk-osx-application-common-gtk3 is meant to be used only in a GTK quartz
development environment but your version of GTK does not
support quartz.  Please make sure that port gtk3 and all its
dependencies are built with variants +quartz and try again.

Error: Failed to configure gtk-osx-application-common-gtk3: gtk3 +quartz not installed.
DEBUG: Error code: NONE
DEBUG: Backtrace: gtk3 +quartz not installed.

@pmetzger
Copy link
Member

Is that the default variant?

cad/gtkwave/Portfile Show resolved Hide resolved
@pmetzger
Copy link
Member

CI is still failing.

@ryandesign
Copy link
Contributor

CI is still failing.

CI will never succeed because this port depends on gtk-osx-application-common-gtk3 which depends on the +quartz variant of gtk3 which is not the default variant. So we cannot rely on CI results when deciding how to proceed with this PR.

@pmetzger
Copy link
Member

Well, then we'll need the submitter to still fix any remaining issues above. @reneeotten are you happy at this point?

Copy link
Contributor

@reneeotten reneeotten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not remove "--disable-silent-rules"

@@ -40,9 +40,6 @@ configure.args-append \
--with-tcl=${prefix}/lib \
--with-tk=${prefix}/lib \
--disable-mime-update \
--disable-silent-rules \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should remain disabled - or is it not a configure argument anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added back the disable-silent-rules.

@reneeotten reneeotten merged commit c5f7df7 into macports:master Feb 5, 2024
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 participants