Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#60612 closed defect (fixed)

virt-manager fails to parse when gtk2 is not active

Reported by: nickgaya (Nick Gaya) Owned by: danchr (Dan Villiom Podlaski Christiansen)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: virt-manager

Description

Travis / AzureCI builds are failing for a recently created PR, seemingly unrelated to my changes.

Example: https://travis-ci.org/github/macports/macports-ports/jobs/695593852

Creating port index in /Users/travis/build/macports/macports-ports
Failed to parse file gnome/virt-manager/Portfile: Registry error: gtk2 not registered as installed & active.
Adding port print/ghostscript
Adding port print/ghostscript-fonts-hiragino

Total number of ports parsed:	3 
Ports successfully parsed:	2 
Ports failed:			1 
Up-to-date ports skipped:	23825

The command "./_ci/bootstrap.sh" failed and exited with 2 during .

Change History (3)

comment:1 Changed 4 years ago by jmroot (Joshua Root)

Component: server/hostingports
Owner: changed from admin@… to danchr
Port: virt-manager added
Status: newassigned
Summary: bootstrap.sh fails when creating port indexvirt-manager fails to parse when gtk2 is not active

The virt-manager port appears to be using the active_variants portgroup incorrectly, resulting in parse failure. I guess bootstrap.sh treats all Portfile parse failures as fatal, and fixing that for the case where the port that fails to parse is not relevant to the PR being tested would mean parsing the output of portindex and figuring out if any of the failures are from Portfiles modified by the PR, or dependencies thereof. Fixing virt-manager is the short-term easier fix.

Last edited 4 years ago by jmroot (Joshua Root) (previous) (diff)

comment:2 Changed 4 years ago by danchr (Dan Villiom Podlaski Christiansen)

Resolution: fixed
Status: assignedclosed

In 09fb5ed1bba624342bfdbc7781c7eeb405cb2050/macports-ports (master):

virt-manager: fix default variant detection

Fixes: #60613
Fixes: #60612

comment:3 Changed 4 years ago by danchr (Dan Villiom Podlaski Christiansen)

I pushed this too soon, but it should be fixed now, sorry!

Note: See TracTickets for help on using tickets.