Skip to content

Conversation

@hbugdoll
Copy link
Contributor

@hbugdoll hbugdoll commented Jan 12, 2026

Introduction:

  • build system's invokeXcodeBuild() add blank lines between Xcode tasks (as in the original xcodebuild log) and colored the first line of a task to make things easier to read
    • done for 23 Xcode task names so far
    • e.g. GenerateAssetSymbols or WriteAuxiliaryFile were missing

Description:

  • completed Xcode task name list in invokeXcodeBuild()
    • now 63 Xcode task names
  • only relevant for log level trace
Before After

@cb1kenobi
Copy link
Contributor

I get what you're trying to do, but this is silly. If it were me, I would completely remove all of the colors.

But I like color too. I get it. I have to believe there's a much simpler solution that works for 90% of the scenarios using a regex that does some whitespace lookbehinds and lookaheads. It doesn't seem that complex.

@hbugdoll
Copy link
Contributor Author

hbugdoll commented Jan 13, 2026

I get what you're trying to do, but this is silly. If it were me, I would completely remove all of the colors.

The colors are just additional highlights (very useful in my opinion in the gray text desert at log level trace).
The main point is to restore the line formatting of the xcodebuild log a bit.

But I like color too. I get it. I have to believe there's a much simpler solution that works for 90% of the scenarios using a regex that does some whitespace lookbehinds and lookaheads. It doesn't seem that complex.

I improved the regex, so that now all that remains is the adding of keywords and the optional space ' ?'.

@hansemannn hansemannn merged commit 5725831 into tidev:main Jan 15, 2026
7 checks passed
@hbugdoll hbugdoll deleted the xcode-log-pretty branch January 15, 2026 08:59
hansemannn pushed a commit that referenced this pull request Jan 29, 2026
* chore(ios): completed Xcode task names for pretty build log

* chore(ios): improved taskRegExp

* chore(ios): removed another space in taskRegExp
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.

3 participants