Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Commit

Permalink
Fix legacy build uitest (#41)
Browse files Browse the repository at this point in the history
* change button title when user wants to change output dir

* fix xcodeproj output dir is not saved to disk

* save delete previous xcodeproj option to disk

* fix recent document sometimes empty

* use modulemap name for PRODUCT_NAME

* bump version

* code sign allowed if legacy build system

* update project

* update

* bump version
  • Loading branch information
wendyliga authored Dec 16, 2021
1 parent 2827d19 commit 2689fae
Show file tree
Hide file tree
Showing 6 changed files with 34 additions and 14 deletions.
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
# CHANGELOG

## 0.7.1
<h2>Tulsi++ 0.7.1 🎅 </h2>
<h4>Bug Fixes:</h4>
<ol>
<li>fix fail building uitest on legacy build</li>
</ol>

----

## 0.7.0
<h2>Tulsi++ 0.7.0 🎅 </h2>
<h4>Bug Fixes:</h4>
Expand Down
3 changes: 3 additions & 0 deletions Tulsi.tulsiproj/Configs/Tulsi.tulsigen
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@
},
"ProjectPrioritizesSwift" : {
"p" : "YES"
},
"UseLegacyBuildSystem" : {
"p" : "YES"
}
},
"projectName" : "Tulsi",
Expand Down
16 changes: 16 additions & 0 deletions Tulsi.tulsiproj/wendy.liga.tulsiconf-user
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{
"optionSet" : {
"BazelPath" : {
"p" : "/opt/homebrew/Cellar/bazelisk/1.10.1/bin/bazelisk"
},
"DeletePreviousXcodeproj" : {
"p" : "YES"
},
"WorkspaceRootPath" : {
"p" : "/Users/wendy.liga/Developer/tulsi-plus-plus"
},
"XcodeprojOutputPath" : {
"p" : "/Users/wendy.liga/Developer/tulsi-plus-plus"
}
}
}
10 changes: 0 additions & 10 deletions Tulsi.tulsiproj/wendyliga.tulsiconf-user

This file was deleted.

8 changes: 5 additions & 3 deletions src/TulsiGenerator/PBXTargetGenerator.swift
Original file line number Diff line number Diff line change
Expand Up @@ -781,7 +781,9 @@ final class PBXTargetGenerator: PBXTargetGeneratorProtocol {
// Bazel takes care of signing the generated applications, so Xcode's signing must be disabled.
buildSettings["CODE_SIGNING_REQUIRED"] = "NO"
// as bazel takes care of signing the application, will silent error when using new build system `An empty identity is not valid when signing a binary for the product type 'Aplication'`
buildSettings["CODE_SIGNING_ALLOWED"] = "NO"
let useLegacyBuildSystem = options[.UseLegacyBuildSystem].commonValueAsBool == true
buildSettings["CODE_SIGNING_ALLOWED"] = useLegacyBuildSystem ? "YES" : "NO"

buildSettings["CODE_SIGN_IDENTITY"] = ""

// Explicitly setting the FRAMEWORK_SEARCH_PATHS will allow Xcode to resolve references to the
Expand Down Expand Up @@ -1683,8 +1685,8 @@ final class PBXTargetGenerator: PBXTargetGeneratorProtocol {
// CODE_SIGNING_REQUIRED=NO so disable code signing and let bazel_build.py do the necessary
// signing.
if pbxTargetType == .AppClip {
buildSettings["CODE_SIGNING_ALLOWED"] = "NO"
buildSettings["CODE_SIGNING_ALLOWED"] = "NO"
let useLegacyBuildSystem = options[.UseLegacyBuildSystem].commonValueAsBool == true
buildSettings["CODE_SIGNING_ALLOWED"] = useLegacyBuildSystem ? "YES" : "NO"
}

// bazel_build.py uses this to determine if it needs to pass the --xcode_version flag, as the
Expand Down
2 changes: 1 addition & 1 deletion version.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
# Version number (recorded into the Info.plist)
TULSI_VERSION_MAJOR = "0"
TULSI_VERSION_MINOR = "7"
TULSI_VERSION_FIXLEVEL = "0"
TULSI_VERSION_FIXLEVEL = "1"

TULSI_VERSION_COPYRIGHT = "2021"
TULSI_PRODUCT_NAME = "Tulsi++"
Expand Down

0 comments on commit 2689fae

Please sign in to comment.