Skip to main content
aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-06-05gcov: Remove useless semicolon too.Alexander Kurtakov2-5/+5
And a bit further save action automation - remove trailing whitespaces. Change-Id: I6fc55db64a6ff31e76ca5eb465be82e5ec769a96 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/123986
2018-06-05gcov: Remove unused imports.Alexander Kurtakov2-4/+60
And enable save actions so this doesn't happen in the future. Change-Id: I3d1673b9038abad3695d922aade569f588daa990 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/123985
2018-06-05Update build machineryAlexander Kurtakov4-6/+6
Update to Tycho 1.2.0, javadoc to 3.0.1, repoanalyzer to latest May I-build. Change-Id: I9c99f5f5acbec8ed587d7abcae13e7446ad2c83c Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/123981 Tested-by: CI Bot
2018-05-31Bug 535421 - AttachLog can stall and cause CDT Container build freezeJeff Johnston1-1/+8
- modify ContainerCommandProcess waitFor() and destroy() methods to use close of attachLog token to kill the log when needed (current method of closing piped streams doesn't work, nor does interrupting thread or trying to stop it) - remove the stack trace print in the log thread and add a new catch clause for any Exception so that when we close the attachLog token we don't get an exception listed in the error log Change-Id: I11051412e86bb1e70daa6798ec08d7b35c81b8a8 Reviewed-on: https://git.eclipse.org/r/123778 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-30Bug 535167 - "Build Image..." run config doesn't accept *.dockerfileJeff Johnston11-17/+102
- change Build Image shortcut to allow .dockerfile extension - save the dockerfile name as a configuration attribute and default to Dockerfile if not found in configuration - change the Build Image config page main tab to show the dockerfile name in addition to the directory - add a new buildImage method to DockerConnection that takes the dockerfile name as a parameter and calls the corresponding docker-client method that allows dockerfile name specification Change-Id: I5a3338ccc5969d0091951d76ca6d3680540aa908 Reviewed-on: https://git.eclipse.org/r/123586 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com> Reviewed-by: Mickael Istria <mistria@redhat.com>
2018-05-29Bug 494296 - GCov plugin handling of linked resourcesJeff Johnston8-18/+186
- modify STSymbolManager and add 3 new methods: (1) findFileFromPath to find linked resource (2) getProjectFromFile to resolve project from file (3) findOneAndOnlyBinary to find binary which can be linked - replace all occurences of getFileForLocation by STSymbolManager.sharedInstance.findFileFromPath Change-Id: I551452e969552e4b5cea7cc2fc5fc0ba46d0063f Reviewed-on: https://git.eclipse.org/r/123109 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-29Fix missing parameter for log message.Jeff Johnston1-2/+2
Change-Id: Iddd97f133a70e8156bf715df66dc81995b64d4e4 Reviewed-on: https://git.eclipse.org/r/123538 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-25Revert change to use attachContainerOutputJeff Johnston1-1/+4
- previous change causes regression to CDT Container build - previous change requires Container to be running at time of attach but we can't guarantee that the Container hasn't already finished in which case we get an exception or possibly miss output - instead key off the requirement of stdin; if specified, assume command is interactive and use attachContainerOutput, otherwise use attachLogs so the command can already be finished Change-Id: I5a7d7d48c456d4dabea7b6dc591708fcf1ab676f Reviewed-on: https://git.eclipse.org/r/123368 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-25Attach to stdout directly instead of attaching to logsMickael Istria2-1/+13
logs are processed with some delays or waiting for newlines. As we don't really have need for logging info in the console and as it can be missing content, let's bind the process directly to stdout stream instead. Change-Id: If273b0318e61666ac033a5afa2b183216bd67f89 Signed-off-by: Mickael Istria <mistria@redhat.com> Reviewed-on: https://git.eclipse.org/r/123351 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-25Modify DockerConnection attachCommand to use operation tokenJeff Johnston3-3/+12
- add a new version of attachCommand to use an operation token - modify ContainerCommandProcess to use the new version of attachCommand similar to how attachLog is used there Change-Id: Id9f2dd29440d73c9b2892796e5199763fa647bd5 Reviewed-on: https://git.eclipse.org/r/123292 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-25Fix warnings in build.properties.Alexander Kurtakov3-3/+5
Change-Id: I4f48fe176532a87dac86fa37d2236d9deeca31f0 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/123130
2018-05-23releng: Remove terminal workaround from target platform.Alexander Kurtakov1-3/+1
Change-Id: I1f59ff42aaf0b9dc9e9b6f76e2281a06fca81e5f Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/123131 Tested-by: CI Bot
2018-05-22Bug 534725 - Cannot run locally built image with Launch ConfigurationJeff Johnston3-1/+20
- add getImage() method to IDockerConnection2 interface - fix getImage() method in DockerConnection to not log an error if image not found, just return null - modify ImageRunSelectionModel validator to add logic if the image name isn't found in the image list, then try and get Image info for the id if the length is >5 and return an IDockerImage if info is found Change-Id: Ieb074a339e7a9eb6dc58c428fa94e4beaeb68ac2 Reviewed-on: https://git.eclipse.org/r/122984 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-22Bug 534683 - Docker Run Image ignores stdin in regular console inputMickael Istria1-3/+46
Change-Id: I464c99d3844d3834e5ade764a390bc4186e53d06 Signed-off-by: Mickael Istria <mistria@redhat.com> Reviewed-on: https://git.eclipse.org/r/122684 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-18Bug 534682 - Terminate in Debug view doesn't work for Docker Run ImageJeff Johnston1-7/+8
- fix ContainerCommandProcess destroy() method to kill the container - remove synchronized specifier as destroy should be able to run at same time as waitFor() - do the same for exitValue() method Change-Id: I191bdff5b40ccd495d7efe43ef396384897a77cc Reviewed-on: https://git.eclipse.org/r/122979 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-17Add Automatic-Module-Name for Java 9 compat.Alexander Kurtakov3-0/+3
Change-Id: I74d8aff924852c7227b818e1447c664b4ad9486f Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122838 Tested-by: CI Bot
2018-05-17rpm: Enable all rpm.core tests.Alexander Kurtakov2-2/+10
And save action to format code. Change-Id: I34c91667de4f75ce647b8d4e07c9aa38c3fdb3dd Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122837 Tested-by: CI Bot
2018-05-14Remove unused suppresswarnings.Alexander Kurtakov5-5/+2
Change-Id: I56d64fc3cbdca10164f98aed1b4a7a103e3c8aa5 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122614
2018-05-14Remove unused imports.Alexander Kurtakov4-6/+0
Change-Id: I525bf46ceff1017f35d55a1ac800e0bbc838d4cb Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122613
2018-05-14Update OSIO mylyn connector license to EPL 2.0.Alexander Kurtakov181-908/+1276
Change-Id: I0ee738789aba5c40064f4875e34bbf0f904dbc78 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122612
2018-05-14Update oprofile plugin license header to EPL 2.0.Alexander Kurtakov118-2445/+2642
Change-Id: I2533fb783246097a4f9f2aa3aa8cb3cef91e658a Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122595
2018-05-14Update perf plugin license headers to EPL 2.0.Alexander Kurtakov72-380/+497
Change-Id: Idf7384ffd1b6e61837662ef7c4e6623a2b9519bb Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122591
2018-05-14Update containers plugin license to EPL 2.0.Alexander Kurtakov540-3220/+3867
Change-Id: Ice4b6d11c51ce6ebb5056a41e504239431324b40 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122586 Tested-by: CI Bot
2018-05-14Update libhover license headers to EPL 2.0.Alexander Kurtakov68-394/+466
Change-Id: I5fe17f11a1fa6fd42d456fde883c37b5bf60e006 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122567
2018-05-14Update rpm plugin license headers to EPL 2.0.Alexander Kurtakov36-263/+251
Properties files this time. Change-Id: I160d67cc93043b236337d69b51413a604b6ad217 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122564 Tested-by: CI Bot
2018-05-14Update rpm plugin license headers to EPL 2.0.Alexander Kurtakov322-8937/+9200
Change-Id: Ie9e7ba603f68b8e4a1aa72628b1457ead29a478e Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122563 Tested-by: CI Bot
2018-05-11Bug 534529 - NullPointerException pulling helloworld imageJeff Johnston1-10/+9
- change DockerProgressDetail class to take Long parameters instead of long as the internal class used to fetch the values is now returning Long instead of long and attempting to convert a null value into long cause the NPE Change-Id: I9b57c991755c6e2ae4e15c2c33e28271e5dc85f8 Reviewed-on: https://git.eclipse.org/r/122498 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
2018-05-11Update gprof license headers to EPL 2.0.Alexander Kurtakov64-334/+457
Change-Id: I5772eb0e0e40d2cbe5953d2dc06bf8c59507f96f Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122499 Tested-by: CI Bot
2018-05-10rpm: Remove .spec form editor.Alexander Kurtakov9-426/+0
It is not finished, doesn't render correctly and doesn't show the most interesting informatation build and install sections. Combined with the overall move towards light weight editors this one is being removed now. Change-Id: I3eb8829ba3a7dd2e50fad13a24a566147fd5c52d Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122438
2018-05-10Update gcov license headers to EPL 2.0.Alexander Kurtakov91-487/+646
Change-Id: I49f7fcaad0609e8fd1e419ab20504e26a13d228d Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122436 Tested-by: CI Bot
2018-05-10Call open() before docker view is used.Rastislav Wagner4-0/+11
Change-Id: Ic89c968902679a3054db7172d2de8b06d9e5377d Signed-off-by: Rastislav Wagner <rawagner@redhat.com> Reviewed-on: https://git.eclipse.org/r/122418 Tested-by: CI Bot Reviewed-by: Alexander Kurtakov <akurtako@redhat.com>
2018-05-10docker: Remove copy of RedDeer WorkbenchView.Alexander Kurtakov8-397/+12
Target platform contains new enough version so this is not needed. Change-Id: Ie1b369d2da272d16cca6a986d2b2b6f6dab8f314 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122402
2018-05-10Add automatic module name for Java 9 compatibility.Alexander Kurtakov2-0/+2
Change-Id: Ia92734f68b4bc15c8e9af0aa76a78d70ffe5b058 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122425
2018-05-10Change license in master pom.Alexander Kurtakov1-7/+6
Change-Id: I1c3624fdac5ad81133e2661976e997770b9d7dff Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122419
2018-05-10Update manpage plugin headers to EPL 2.0.Alexander Kurtakov25-125/+171
Change-Id: I33651358d5abf5a4b011a625264bba71519eb7da Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122417
2018-05-10Update javadocs plugin headers to EPL 2.0.Alexander Kurtakov19-103/+135
Change-Id: I8729c159193c6b7daf6711d17cddd9ae9798d231 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122416
2018-05-10Update releng headers to EPL 2.0Alexander Kurtakov9-70/+74
Change-Id: I8d7289e5a37db4489dc0167bab763788b108d9c3 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122415 Tested-by: CI Bot
2018-05-10docker: Convert to try-with-resources.Alexander Kurtakov8-208/+188
Change-Id: I361a0664f915be896a7e7d62f96cc9153b8d52e3 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122404
2018-05-10Change systemtap license headers to EPL 2.0.Alexander Kurtakov384-1980/+2678
Change-Id: I0bb445f0215f48a730a8efdd7ae1bcbadab477e4 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122397
2018-05-10Update changelog plugin headers to EPL 2.0.Alexander Kurtakov81-511/+565
Change-Id: Ibc4d9214a1a65cdcd76a032524bd9fe44cccbc57 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122390
2018-05-10Update Vagrant license headers to EPL 2.0.Alexander Kurtakov63-326/+439
Change-Id: Icd1dbbb898262f28cd992eae00f7c854da165ee5 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122389
2018-05-10valgrind: Another change to make line numbers in tests match.Alexander Kurtakov1-2/+0
Change-Id: Ibfc6ec1ff2b54c21f74b93f8bd998cd2097cc0ac Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122385 Tested-by: CI Bot
2018-05-10valgrind: Change cpp test file so line numbers match.Alexander Kurtakov1-2/+0
Change-Id: I3e10b52cbcfd14aeffe0b548c538bc343fb2acd9 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122383
2018-05-10gcov: Declaration and definitio at the same line.Alexander Kurtakov1-3/+2
Change-Id: I6664aa104c9f06721113375657136bac97236af4 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122382
2018-05-09Update maven plugins.Alexander Kurtakov1-4/+3
Resources, cbi and site. Change-Id: I0f226e5be90ed875fec5d432b6fe245b27a37b5f Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122384
2018-05-09Update valgrind metainfo file to point to EPL 2.0 too.Alexander Kurtakov1-6/+7
Change-Id: Ic2106f68e54b8a22707bfff58f6a8b44d79856e4 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122379
2018-05-09Remove useless throws declarations in docker tests.Alexander Kurtakov1-2/+3
Change-Id: I72947cf435034e137be2f7ec1fe7f37165fb1dcf Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122378
2018-05-09Change valgrind license headers to EPL 2.0.Alexander Kurtakov210-1216/+1617
Change-Id: Iaee1b118089379642eecb19be18e39333b4316ff Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122377
2018-05-09Chance license header ot EPL 2.0 in profiling dir.Alexander Kurtakov206-1033/+1456
Change-Id: I831ff4b7bafbf726f061596b5328dae9cb061d7d Signed-off-by: Alexander Kurtakov <akurtako@redhat.com> Reviewed-on: https://git.eclipse.org/r/122373
2018-05-07Bug 534367 - have RunDockerImage Delegate attach an IProcessJeff Johnston6-14/+40
- fix DockerConnection startContainer() methods to ignore error 304 which means the Container is already started - have RunConsole attachToTerminal check out stream for null before using to notify - change RunImageCommandHandler runCommand() to accept an ILaunch parameter and to create a ContainerCommandProcess to start the Container. If the ILaunch parameter is non-null, then wrap the ContainerCommandProcess with a RuntimeProcess which will be registered with the ILaunch - pass ILaunch to RunImageCommandHandler runImage() method from RunDockerImageLaunchConfigurationDelegate - modify ContainerCommandProcess constructor to take an extra OutputStream parameter to pass to the startContainer() call - in ContainerCommandProcess waitFor() method verify there are remoteVolumes before creating a CopyVolumesJob - fix bug in ContainerLauncher that tries to reference invalid char position when volume equals dir - when creating a new ContainerCommandProcess in ContainerLauncher, pass null OutputStream as extra parameter so startContainer() will use default console Change-Id: I2a07a529019a34dd32e65160cf74b4db5a3bfbce Reviewed-on: https://git.eclipse.org/r/122282 Tested-by: CI Bot Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>

Back to the top