can u compile it?

# announcements

[qwert_ukg]

can u compile it?

[holgerbrandl]

No Error:(2, 7) Kotlin: Unresolved reference: DependsOn

To see yourself @Czar @qwert_ukg just add compile 'com.github.holgerbrandl:kscript-annotations:1.1' to a project and add a @file:DependsOn("de.mpicbg.scicomp:kutils:0.4") somewhere before a package statement.

[holgerbrandl]

Maybe it’s a compiler bug?

[Czar]

How is DependsOn defined?

[Czar]

Its target should be AnnotationTarget.FILE, check here: https://kotlinlang.org/docs/reference/annotations.html

[holgerbrandl]

It is, see https://github.com/holgerbrandl/kscript-annotations/blob/master/src/main/kotlin/ScriptDirectives.kt#L5

[holgerbrandl]

The problem actually can be reproduced more easily by creating a annotation containing class in the toplevel of a project.

[holgerbrandl]

To prove that point I’ve created a small test-repo under
https://github.com/holgerbrandl/file_target_annotations

[holgerbrandl]

@Czar What do you think?

[Czar]

Oh, you have the annotation in different package than your file, you just need to import, and by the way it is really bad idea to use default package. To fix specifically your example add import MyAnno after package declaration.

[Czar]

You're right that IJ has some problem here though, it should've suggested to import the annotation. Although if you ask me I'd leave this bug unreported to dissuade people from putting stuff in root package 😄

[holgerbrandl]

Thanks a lot for the nice explanation. I incorrectly assumed that the default package would be inferred from the source root directory structure, but in fact kotlin assumes the root package as default, if no package is specified. That’s why the import MyAnno is not need without the package statement.

[holgerbrandl]

I agree about avoiding the root package for the annotations implementation.

[Czar]

You're welcome, you can look up Kotlin docs on the matter here: https://kotlinlang.org/docs/reference/packages.html
It's quite brief, so won't take much time.

[holgerbrandl]

Still the import should work imho so https://youtrack.jetbrains.com/issue/KT-21630