Merge pull request #247 from alexarchambault/topic/properties-special-characters

Don't restrict Maven property name characters too much
This commit is contained in:
Alexandre Archambault 2016-05-10 16:00:13 +02:00
commit bb77e85f1e
2 changed files with 5 additions and 5 deletions

View File

@ -71,7 +71,7 @@ object Resolution {
}
val propRegex = (
quote("${") + "([a-zA-Z0-9-.]*)" + quote("}")
quote("${") + "([^" + quote("{}") + "]*)" + quote("}")
).r
def substituteProps(s: String, properties: Map[String, String]) = {

View File

@ -34,11 +34,11 @@ object ResolutionTests extends TestSuite {
Project(Module("acme", "play"), "2.4.1",
dependencies = Seq(
"" -> Dependency(Module("acme", "play-json"), "${playJsonVersion}"),
"" -> Dependency(Module("${project.groupId}", "${configName}"), "1.3.0")),
"" -> Dependency(Module("acme", "play-json"), "${play_json_version}"),
"" -> Dependency(Module("${project.groupId}", "${WithSpecialChar©}"), "1.3.0")),
properties = Seq(
"playJsonVersion" -> "2.4.0",
"configName" -> "config")),
"play_json_version" -> "2.4.0",
"WithSpecialChar©" -> "config")),
Project(Module("acme", "play-extra-no-config"), "2.4.1",
Seq(