hrev52946 adds 1 changeset to branch 'master'
old head: 2c73537c43dbd66980e4f2595eea85dcf5925969
new head: c91b716e09f9b25e02c4827d08c92d11d5c9ff6c
overview:
https://git.haiku-os.org/haiku/log/?qt=range&q=c91b716e09f9+%5E2c73537c43db
----------------------------------------------------------------------------
c91b716e09f9: HaikuDepot : Generate Sources -Fix Build
In recent changes to HD's build files to generate
some sources at build-time there was a problem
with builds that were configured to run with some
element of concurrency. In this change, the steps
that should be in sequence are done with shell
commands to avoid race conditions.
Change-Id: I5b6fb4907d8ea5f3ca90de956ecce322cd89c685
Reviewed-on: https://review.haiku-os.org/c/1130
Reviewed-by: Adrien Destugues <pulkomandy@xxxxxxxxx>
[ Andrew Lindesay <apl@xxxxxxxxxxxxxx> ]
----------------------------------------------------------------------------
Revision: hrev52946
Commit: c91b716e09f9b25e02c4827d08c92d11d5c9ff6c
URL: https://git.haiku-os.org/haiku/commit/?id=c91b716e09f9
Author: Andrew Lindesay <apl@xxxxxxxxxxxxxx>
Date: Sat Mar 2 10:09:32 2019 UTC
----------------------------------------------------------------------------
2 files changed, 5 insertions(+), 7 deletions(-)
src/apps/haikudepot/build/jam/HdsSchemaGenRules | 10 ++++------
src/apps/haikudepot/model/Model.cpp | 2 +-
----------------------------------------------------------------------------
diff --git a/src/apps/haikudepot/build/jam/HdsSchemaGenRules
b/src/apps/haikudepot/build/jam/HdsSchemaGenRules
index 8d2100b730..c72a3e15e2 100644
--- a/src/apps/haikudepot/build/jam/HdsSchemaGenRules
+++ b/src/apps/haikudepot/build/jam/HdsSchemaGenRules
@@ -39,15 +39,14 @@ rule HdsSchemaGenModel
Clean $(1:D) ;
Depends $(1) : $(2) $(3) ;
-
- MkDir $(1:D) ;
- HdsSchemaGenTouch $(1) ;
HdsSchemaGenModel1 $(1) : $(2) $(3) $(1:D) ;
}
actions HdsSchemaGenModel1
{
+ mkdir -p $(2[3])
python $(2[2]) -i $(2[1]) --outputdirectory $(2[3])
+ touch $(1)
}
# pragma mark - Bulk Parsing Class Generation
@@ -63,15 +62,14 @@ rule HdsSchemaGenBulkParser
Clean $(1:D) ;
Depends $(1) : $(2) $(3) ;
-
- MkDir $(1:D) ;
- HdsSchemaGenTouch $(1) ;
HdsSchemaGenBulkParser1 $(1) : $(2) $(3) $(1:D) ;
}
actions HdsSchemaGenBulkParser1
{
+ mkdir -p $(2[3])
python $(2[2]) -i $(2[1]) --outputdirectory $(2[3])
--supportbulkcontainer
+ touch $(1)
}
# pragma mark - Registering Generated Classes
diff --git a/src/apps/haikudepot/model/Model.cpp
b/src/apps/haikudepot/model/Model.cpp
index 6b77a2bfe6..f6ee6a675f 100644
--- a/src/apps/haikudepot/model/Model.cpp
+++ b/src/apps/haikudepot/model/Model.cpp
@@ -406,7 +406,7 @@ Model::Model()
// Force the preferred language to one of the currently
supported
// ones, until the web application supports all ISO language
codes.
printf("User preferred language '%s' not currently supported, "
- "defaulting to 'en'.", fPreferredLanguage.String());
+ "defaulting to 'en'.\n", fPreferredLanguage.String());
fPreferredLanguage = "en";
}
fWebAppInterface.SetPreferredLanguage(fPreferredLanguage);