Skip to content

Commit 2b7b842

Browse files
fix: remove spurious NamespacedResourceStore from TestSkin, fix KiaiHitExplosion null Drawable, fix iOS xcode-select
Agent-Logs-Url: https://github.com/winnerspiros/osu/sessions/04bea131-bdfb-4c81-8a1a-dbba5215f82e Co-authored-by: winnerspiros <1675249+winnerspiros@users.noreply.github.com>
1 parent f8f8a13 commit 2b7b842

3 files changed

Lines changed: 18 additions & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -284,12 +284,10 @@ jobs:
284284
# Fix: MacOSX.sdk in Xcode 26.5 is a minimal stub. Replace it with a symlink to the
285285
# real versioned SDK found across Xcode installs. Real SDKs have hundreds of headers;
286286
# stubs have ≤1.
287-
ACTIVE_XCODE="/Applications/Xcode_26.5.app"
288-
sudo xcode-select -switch "$ACTIVE_XCODE"
289-
SDKS_DIR="$ACTIVE_XCODE/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs"
290-
MACOS_SDK="$SDKS_DIR/MacOSX.sdk"
287+
PREFERRED_XCODE="/Applications/Xcode_26.5.app"
291288
REAL_SDK=""
292-
# Search ALL Xcode_26.x.app installs (including active) for versioned MacOSX[N].sdk.
289+
REAL_XCODE=""
290+
# Search ALL Xcode_26.x.app installs for versioned MacOSX[N].sdk.
293291
# Validate by usr/include header count: stubs ≤1 file, real SDKs have hundreds.
294292
for xapp in $(ls -d /Applications/Xcode_26.*.app 2>/dev/null | sort -rV); do
295293
sdk_dir="$xapp/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs"
@@ -299,6 +297,7 @@ jobs:
299297
hdr_count=$(ls "$cand/usr/include" 2>/dev/null | wc -l | tr -d ' ')
300298
if [ "${hdr_count:-0}" -gt 5 ]; then
301299
REAL_SDK="$cand"
300+
REAL_XCODE="$xapp"
302301
break 2
303302
fi
304303
done
@@ -314,7 +313,18 @@ jobs:
314313
fi
315314
done
316315
fi
316+
# Switch to preferred Xcode if it exists, otherwise the one that has the real SDK.
317+
if [ -d "$PREFERRED_XCODE" ]; then
318+
sudo xcode-select -switch "$PREFERRED_XCODE"
319+
elif [ -n "$REAL_XCODE" ]; then
320+
sudo xcode-select -switch "$REAL_XCODE"
321+
echo "Note: Xcode_26.5.app not found; using $REAL_XCODE for xcode-select"
322+
fi
323+
# Replace stub MacOSX.sdk in the active Xcode with a symlink to the real versioned SDK.
317324
if [ -n "$REAL_SDK" ]; then
325+
ACTIVE_DEV="$(xcode-select -p)"
326+
ACTIVE_SDKS_DIR="$ACTIVE_DEV/Platforms/MacOSX.platform/Developer/SDKs"
327+
MACOS_SDK="$ACTIVE_SDKS_DIR/MacOSX.sdk"
318328
sudo rm -rf "$MACOS_SDK"
319329
sudo ln -sfn "$REAL_SDK" "$MACOS_SDK"
320330
echo "Created MacOSX.sdk symlink -> $REAL_SDK"

osu.Game.Rulesets.Catch.Tests/TestSceneHyperDashColouring.cs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
using osu.Framework.Graphics;
1111
using osu.Framework.Graphics.Containers;
1212
using osu.Framework.Graphics.Sprites;
13-
using osu.Framework.IO.Stores;
1413
using osu.Framework.Testing;
1514
using osu.Game.Beatmaps;
1615
using osu.Game.Beatmaps.ControlPoints;
@@ -211,11 +210,7 @@ public Colour4 HyperDashFruitColour
211210
}
212211

213212
public TestSkin()
214-
: base(new SkinInfo(), null,
215-
new NamespacedResourceStore<byte[]>(
216-
new DllResourceStore(typeof(TestSceneHyperDashColouring).Assembly),
217-
@"Resources/special-skin"),
218-
string.Empty)
213+
: base(new SkinInfo(), null, null, string.Empty)
219214
{
220215
}
221216
}

osu.Game.Rulesets.Taiko/UI/KiaiHitExplosion.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ public partial class KiaiHitExplosion : Container
2323

2424
private SkinnableDrawable skinnable = null!;
2525

26-
public override double LifetimeStart => skinnable.Drawable.LifetimeStart;
26+
public override double LifetimeStart => skinnable?.Drawable?.LifetimeStart ?? base.LifetimeStart;
2727

28-
public override double LifetimeEnd => skinnable.Drawable.LifetimeEnd;
28+
public override double LifetimeEnd => skinnable?.Drawable?.LifetimeEnd ?? base.LifetimeEnd;
2929

3030
public KiaiHitExplosion(DrawableHitObject judgedObject, HitType hitType)
3131
{

0 commit comments

Comments
 (0)