A while ago we made the "Make sure all versions have been bumped
appropriately compared to the baseline" output to another file
to make it clearer. This refactor splits it up into different
build steps instead so that I don't have to open an additional
log file.
This commit runs GitHub actions for code cleanliness.
Because this includes compare and replace build,
this verifies that the commit is buildable and
clean.
The natives are not rebuilt here (yet) because
the GitHub actions runner does not have the
cross compiler tools installed.
Part of migration to GitHub - Issue #32
List as many as possible of the failed bundles when doing the baseline
compare.
Change-Id: I0d2817c418cf865f7fc6ea1c7e25d5b3b7b41bd5
Signed-off-by: Torbjörn Svensson <azoff@svenskalinuxforeningen.se>
As more checks have been added it has gotten a little unweildy.
The new script "do_all_code_cleanups.sh" now does most of the code
formatting checks.
Change-Id: I65fd888c88e1d7e470ed5621cee18ff3bf6680b4
Because we build cross platform, we want to make sure no unexpected
dependencies are introduced on newer or non-universal libraries
in the native code.
Change-Id: Ib8ffeb5f374bcd04514c31dca9d8297fed70343a
Also generate JNI header for serial component. This component does not
really need it since the implementation is in one file only, but this
would at least throw an error if the java part changes and the native
part is not updated accordingly.
Change-Id: Id598410c322580bdda37c905ed08627390c913ba
Signed-off-by: Torbjörn Svensson <azoff@svenskalinuxforeningen.se>
The native directory can contain both libraries and utilities. In
Gerrit https://git.eclipse.org/r/c/cdt/org.eclipse.cdt/+/165270,
there is a utility, so the directory name "jni", or even "library" does
not work. In order to support both utilities and libraries, the
suggested name of the directory is "native_src" and thus, this commit
syncs that change in o.e.c.native.serial.
Change-Id: Iafa9ce9ae1dca7ef563ab397e8b3eb0b8642372d
Signed-off-by: Torbjörn Svensson <azoff@svenskalinuxforeningen.se>
When git is configured to honor executable bit for files
(core.fileMode=true), the .exe and .dll files in the
repository needs to have execute bit set to be able to run
Eclipse in runtime mode on Windows.
For more details:
https://git-scm.com/docs/git-config#Documentation/git-config.txt-corefileMode
"Git for Windows" does not appear to have this problem,
but at least Cygwin does.
Change-Id: I4c164f6d99219d461c301189f101ccbf63debb50
Signed-off-by: Torbjörn Svensson <azoff@svenskalinuxforeningen.se>
This commit on its own fails cleanliness, but is separated out
to make review easier. The subsequent commit is the result of
cleaning the code with the script
Change-Id: Ibaddc2f338f14fbe60662acc8cb272b584a1a828
Expand the copy mask for .settings/org.eclipse.pde.*, copy only
$COREPROJECT/.settings/org.eclipse.pde.prefs
$COREPROJECT/.settings/org.eclipse.pde.api.tools.prefs
Change-Id: I54d3bb65bb8b2a6d1054bff84fd6185e3fec67ef
Signed-off-by: Alexander Fedorov <alexander.fedorov@arsysop.ru>
Now that this script does not need to be in a common place
for launchbar and tools.templates bring it back into the main
repo.
Change-Id: Ia3033a44bbb809e9163b8cf6b14b49e63f151dd1
The check-code-cleanliness CI jobs have not been running recently,
so this commit is to the code up to those minor changes
Change-Id: I9fb4b0213dec558f58c5ca718b3d2685564ef456
The DSF Examples contains some commented out source that is enabled
via a script so this commit cleans that up for new formatting rules.
Change-Id: I82c5e195cc3746415c659b2c0fc72b8118cdc56c
The previous alignment of all the warnings/ignores
led to too many warnings that weren't there before. This
commit relaxes them a bit.
The core/org.eclipse.cdt.core/.settings/org.eclipse.jdt.core.prefs
is still the "master" copy, with
releng/scripts/check_code_cleanliness.sh containing the
exceptions that apply to test plug-ins.
Change-Id: Ibd4e31ade0b42b31e7cbe5a94f06c6fc15183a56
git ls-files has two benefits:
- Only finds tracked files
This is necessary because we were having a few cleanups
being attempted on ignored files.
- Is much faster
Change-Id: I5a53497f8ef9c839676b7ef7e3be35171140f4b9
