remove InetAddress.getLocalHost().hostName call [MERGE_WHEN_READY] #88
Conversation
…e seconds, and affected every configuration; probably only on mac: https://thoeni.io/post/macos-sierra-java/ my machine: impact plugin apply 5000ms+ -> 70ms
dsvoronin
changed the title
|
|
||
| return hostName.startsWith("android-builder") | ||
| private fun isRunViaMirakle(project: Project): Boolean { | ||
| return project.hasProperty("mirakle.build.on.remote") |
There was a problem hiding this comment.
где проставляем это свойство?
There was a problem hiding this comment.
There was a problem hiding this comment.
проверил в интеграции руками, свойство видно изнутри билда
| @@ -54,10 +54,8 @@ fun isRunFromGradleTestKit(project: Project): Boolean { | |||
| fun isRunInGradleTestKit(): Boolean = | |||
| System.getProperty("isTest", "false") == "true" | |||
|
|
|||
| private fun isRunInMainframer(): Boolean { | |||
| val hostName = InetAddress.getLocalHost().hostName | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
Как ты наткнулся на это?
There was a problem hiding this comment.
это выглядит как подвисающая кофигурация, когда внутри пытаемся резолвить localhost
|
мда... |
ну в целом оно и не нужно, это единственный вызов в узком месте у нас |
|
Описал причины, почему нужно самим отлавливать и не ходить по минному полю. |
| * IDEA добавляет специальный init script, по нему понимаем что запустили в IDE | ||
| * используется в :test-project | ||
| */ | ||
| systemProperty("isInvokedFromIde", |
There was a problem hiding this comment.
Как ты узнал про это?
А где почитать про концепцию инитскриптов?
There was a problem hiding this comment.
There was a problem hiding this comment.
узнал просто залогировав параметры на старте, из консоли и из IDE различаются немного, это позволило сделать такую проверку
она не супер грамотная, т.к. только JB IDE детектит, но у меня не было цели все закрыть
could take 5+ seconds, and affects every configuration;
probably only on mac: https://thoeni.io/post/macos-sierra-java/
my machine: impact plugin apply 5000ms+ -> 70ms