-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Resolving doesn't work #141
Comments
Thank-you for trying out the library. Sorry you are not finding the library working as expected. I am not much of an expert on using the library so if you could provide some minimized code so I could test it out that would really help. Another option is to try the lightbend config library with the JVM to see if it works in that environment. I have not been able to follow all the changes they have made so it could be fixed in that library but not this one. |
This is a minimal example. Loaded with val config2 = {
ConfigFactory
.parseString(
new String(
Files.readAllBytes(filesPath.resolve("parameters_base.conf"))
)
)
.resolve()
}
println(config2.getString("basedOnEnv")) Stack trace is:
I can try that tomorrow if you'd like, but I imagine resolving works there |
@markehammons Update: while porting tests to Scala Native, I ran across this that failed only only Scala Native. [error] Test org.ekrich.config.impl.PropertiesTest.makeListWithNumericKeysAndMerge failed: org.ekrich.config.ConfigException$UnresolvedSubstitution: String: 5: Could not resolve substitution to a value: ${a} was part of a cycle of substitutions involving ${a}, ${a}, took 0.004 sec |
@markehammons If you are following, I determined the issue. We will have to wait for a new release of Scala Native and then this PR where I extend the code coverage can be merged and then we can release another version of |
glad to hear it. I haven't been using scala-native much (mainly because
scala 3 has spoiled me), but I'm looking forward to its changes, and
hopefully the project I'm currently working on can be used to bridge
between scala native and scala on the jvm.
…On Sat, Oct 30, 2021 at 3:40 AM Eric K Richardson ***@***.***> wrote:
@markehammons <https://github.com/markehammons> If you are following, I
determined the issue. java.util.IdentityHashMap needed fixing on Scala
Native. Luckily I created a good version on Scala.js that I could port
over. I put in a pull request there. scala-native/scala-native#2425
<scala-native/scala-native#2425>
We will have to wait for a new release of Scala Native and then this PR
where I extend the code coverage can be merged and then we can release
another version of sconfig. Kind of a long path but good either way. #205
<#205>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#141 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAT5CP3K3KPBL5KCAD2GWQLUJNEIDANCNFSM4WVF6TEA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Closed via this PR: #205 So using Scala Native |
I have a config like so:
I merge it with a programmatically generated config
and call resolve, but no matter what I do I get the following error:
The text was updated successfully, but these errors were encountered: