-
Bug
-
Resolution: Fixed
-
P4
-
9
-
b49
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8084441 | emb-9 | Erik Joelsson | P4 | Resolved | Fixed | team |
From Magnus' review mail:
*** Issues with source files moving and its repercussions ***
* When the source code has moved, especially the native libraries, most of the specific INCLUDE and EXCLUDE statements are, or should be, unnecessary. Nevertheless, there are still several occasions of such statements. In some cases, it seems like dead code that can (and should) be removed. But in some cases, I believe it is an indication that the source code has still not yet been moved to a suitable location. I believe the end goal with this shuffling regarding native library source code is that there should be a one-to-one correspondance between compiled native library and source code directory. This is now indeed the case for 99% of the libraries, but there are still some exceptions.
This is a slightly vague point at the moment. I indent to check the INCLUDE and EXCLUDE statements more fully and will post a second review with results of what I find. Nevertheless, I think it is important to make sure we do get things correct this time.
---
This will quite possibly result in additional file moves to a more uniform file structure.
*** Issues with source files moving and its repercussions ***
* When the source code has moved, especially the native libraries, most of the specific INCLUDE and EXCLUDE statements are, or should be, unnecessary. Nevertheless, there are still several occasions of such statements. In some cases, it seems like dead code that can (and should) be removed. But in some cases, I believe it is an indication that the source code has still not yet been moved to a suitable location. I believe the end goal with this shuffling regarding native library source code is that there should be a one-to-one correspondance between compiled native library and source code directory. This is now indeed the case for 99% of the libraries, but there are still some exceptions.
This is a slightly vague point at the moment. I indent to check the INCLUDE and EXCLUDE statements more fully and will post a second review with results of what I find. Nevertheless, I think it is important to make sure we do get things correct this time.
---
This will quite possibly result in additional file moves to a more uniform file structure.
- backported by
-
JDK-8084441 Cleanup include and exclude of core-libs native libraries after source code restructure
- Resolved
- relates to
-
JDK-8071469 Cleanup include and exclude of sound native libraries after source code restructure
- Resolved
-
JDK-8071470 Cleanup include and exclude of awt native libraries after source code restructure
- Closed