Skip to content

DRAFT Compile new files #79

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

Draft
wants to merge 17 commits into
base: wix-master
Choose a base branch
from
Draft

DRAFT Compile new files #79

wants to merge 17 commits into from

Conversation

mildagle
Copy link

No description provided.

@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 380a8e9 to 60b5109 Compare February 4, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from 150a6fd to f14b17f Compare February 14, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 207d649 to ef463dd Compare February 17, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 4 times, most recently from 187874c to 269d8f2 Compare February 28, 2023 05:06
ZachiNachshon and others added 14 commits March 1, 2023 05:09
…47)

* only filters labels according to parent packages for LanguageClass.C
This is to keep supporting CPP need while also supporting aggregate targets
More info:
https://bazelbuild.slack.com/archives/CM8JQCANN/p1566481600003600
Filter was deliberately added to fix an issue with C++ headers.
The original bug was that opening the BUILD file for foo/bar.h caused it to open baz/qux/BUILD because there’s a cc_library target in baz/qux that includes foo/bar.h in its headers. The source-to-target look up queries the index for the target during the last sync which compiled the source file. In this case, it was the cc_library target in baz/qux.
So the fix was to ensure that the BUILD file is always for the parent package (foo/bar)
Don't use ijars as they break source navigation

- ignore java_import /_ijar/ as its original location is not know
- replace -ijar with sibling full jar
- replace -hjar with sibling full jar

https://youtrack.jetbrains.com/issue/SCL-16975/Navigation-to-attached-Scala-sources-for-interface-jar-does-not-work
Add support for scala source attachment
Add source root detection for Java and Scala source jars. Detection is required when dire4ctory structure in source jar has deeper structure than source packages.

Source attachement changes for 2022.3 support
`idea.plugin.protoeditor` needs to be aware of source roots of proto
libraries

Add proto files to target sources
Via running bloop and BSP

Fast test supports ITs
Location substitution supports runpath(s) in addition to location(s)
Still includes a hack of the env variable of BUILD_TOOL which we need to see how to remove but at least a smaller hack than before

Full Compile on proto file changes

don't update sources after proto change already exists
@WixBuildServer WixBuildServer force-pushed the wix-master branch 4 times, most recently from dca26e5 to 1995379 Compare May 21, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from efe1103 to cea2706 Compare May 28, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 2 times, most recently from ce912f2 to 004a3d4 Compare June 4, 2025 05:02
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from c330da5 to be9c894 Compare June 14, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 2 times, most recently from 9ffcb81 to 232e0b8 Compare June 19, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 6 times, most recently from d9b89c0 to a89b9fc Compare July 1, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from 811b920 to 8e33bcb Compare July 11, 2025 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants