[JIRA] (JENKINS-4369) Workspace relocation check false positivities

5 views
Skip to first unread message

scm_issue_link@java.net (JIRA)

unread,
Aug 28, 2016, 10:39:02 AM8/28/16
to jenkinsc...@googlegroups.com
SCM/JIRA link daemon commented on Bug JENKINS-4369
 
Re: Workspace relocation check false positivities

Code changed in jenkins
User: Joseph
Path:
.gitignore
pom.xml
src/main/java/hudson/plugins/accurev/AccurevChangeLogSet.java
src/main/java/hudson/plugins/accurev/AccurevLauncher.java
src/main/java/hudson/plugins/accurev/AccurevMode.java
src/main/java/hudson/plugins/accurev/AccurevSCM.java
src/main/java/hudson/plugins/accurev/AccurevStream.java
src/main/java/hudson/plugins/accurev/AccurevTransaction.java
src/main/java/hudson/plugins/accurev/ByteArrayStream.java
src/main/java/hudson/plugins/accurev/CheckForChanges.java
src/main/java/hudson/plugins/accurev/DetermineRemoteHostname.java
src/main/java/hudson/plugins/accurev/FindAccurevClientExe.java
src/main/java/hudson/plugins/accurev/ParseChangeLog.java
src/main/java/hudson/plugins/accurev/PopulateStreams.java
src/main/java/hudson/plugins/accurev/PurgeWorkspaceContents.java
src/main/java/hudson/plugins/accurev/PurgeWorkspaceOverlaps.java
src/main/java/hudson/plugins/accurev/XmlConsolidateStreamChangeLog.java
src/main/java/hudson/plugins/accurev/XmlParserFactory.java
src/main/java/hudson/plugins/accurev/cmd/ChangeLogCmd.java
src/main/java/hudson/plugins/accurev/cmd/Command.java
src/main/java/hudson/plugins/accurev/cmd/History.java
src/main/java/hudson/plugins/accurev/cmd/JustAccurev.java
src/main/java/hudson/plugins/accurev/cmd/Login.java
src/main/java/hudson/plugins/accurev/cmd/PopulateCmd.java
src/main/java/hudson/plugins/accurev/cmd/SetProperty.java
src/main/java/hudson/plugins/accurev/cmd/ShowDepots.java
src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java
src/main/java/hudson/plugins/accurev/cmd/Update.java
src/main/java/hudson/plugins/accurev/delegates/AbstractModeDelegate.java
src/main/java/hudson/plugins/accurev/delegates/ReftreeDelegate.java
src/main/java/hudson/plugins/accurev/delegates/Relocation.java
src/main/java/hudson/plugins/accurev/delegates/SnapshotDelegate.java
src/main/java/hudson/plugins/accurev/delegates/StreamDelegate.java
src/main/java/hudson/plugins/accurev/delegates/WorkspaceDelegate.java
src/main/java/hudson/plugins/accurev/parsers/output/ParseInfoToLoginName.java
src/main/java/hudson/plugins/accurev/parsers/output/ParseLastFewLines.java
src/main/java/hudson/plugins/accurev/parsers/output/ParseOutputToFile.java
src/main/java/hudson/plugins/accurev/parsers/output/ParsePopulate.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseGetConfig.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseHistory.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseLsRules.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseRefTreeExternalFile.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseShowReftrees.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseShowStreams.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseShowWorkspaces.java
src/main/java/hudson/plugins/accurev/parsers/xml/ParseStatOverlaps.java
src/main/java/hudson/plugins/jetty/security/Password.java
src/main/resources/hudson/plugins/accurev/AccurevChangeLogSet/digest.jelly
src/main/resources/hudson/plugins/accurev/AccurevChangeLogSet/index.jelly
src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly
src/main/resources/hudson/plugins/accurev/AccurevSCM/global.jelly
src/main/resources/index.jelly
http://jenkins-ci.org/commit/accurev-plugin/53d62927bc9a0a07a3c2a286ab3b3d1e89a183b5
Log:
Code cleanup, Jenkins-4369 FIX, Log spamming, Javadoc (#25)

  • ignore intellij
  • fixed failing test

Test needed jelly escape by default set to true

  • Override not allowed on interface method

Override is not allowed when implementing interface method

  • JENKINS-4369 Fix - Workspace relocation check false positivities

https://github.com/jenkinsci/accurev-plugin/pull/11

  • Spamming log
  • Uncessary cast
  • Unused import
  • Redundant modifier
  • Unnecessary boxing
  • for loop instead of while.
  • Redundant modifier
  • First stab at updating to 2.7 LTS

Still need a few more deprecation warning to go away.

  • Fix unchecked
  • Try with Resource
  • Collapsed catch
  • Forgot to remove redundant cast
  • Diamond types
  • Code cleanup and unused imports
  • Proper comment
  • First fix to JavaDoc
  • Javadoc only
  • Findbugs: Garbage collection call it bad
  • FindBug: Method call passes null for non-null parameter
  • FindBugs: Reliance on default encoding
  • more idea ignore
  • All FindBugs, fixed.
  • Update .gitignore
  • Target java 1.8
  • FindBugs null pointer

Possible null pointer dereference due to return value of called method
Also optimise import

  • Some more and some less null pointer

Please pass already :smile:

  • FindBug dance :tada:
  • deprecation warnings cleared, some more code cleanup.

Tested on Jenkins 2.7

  • Bumping to 2.13 jenkins plugin

Last touch :tada:

  • Last one, installation path on macOS :apple:

MacOS default installation with the anywhere installer is now on
Applications/AccuRev.

  • Unused dependency?
  • unused import
  • bump plugin versions
  • update maintainer
  • prepare for 0.7.0 release
Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)
Atlassian logo

scm_issue_link@java.net (JIRA)

unread,
Aug 28, 2016, 11:10:01 AM8/28/16
to jenkinsc...@googlegroups.com

Code changed in jenkins
User: Joseph 'Casz' Petersen
Path:
src/main/java/hudson/plugins/accurev/delegates/ReftreeDelegate.java
src/main/java/hudson/plugins/accurev/delegates/WorkspaceDelegate.java
http://jenkins-ci.org/commit/accurev-plugin/c825f7c2d4ca68630ae9e6e6517f080e7729031d
Log:


JENKINS-4369 Fix - Workspace relocation check false positivities

josephp90@gmail.com (JIRA)

unread,
Aug 29, 2016, 9:15:06 AM8/29/16
to jenkinsc...@googlegroups.com
Joseph Petersen resolved as Fixed
 

This should be resolved 0.7.0.

Jenkins / Bug JENKINS-4369
Change By: Joseph Petersen
Status: In Progress Resolved
Assignee: Anton Feoktistov Joseph Petersen
Resolution: Fixed

josephp90@gmail.com (JIRA)

unread,
Mar 7, 2020, 1:08:13 AM3/7/20
to jenkinsc...@googlegroups.com
Joseph Petersen assigned an issue to Joseph Petersen
Change By: Joseph Petersen
Assignee: Joseph Petersen (old)
This message was sent by Atlassian Jira (v7.13.12#713012-sha1:6e07c38)
Atlassian logo
Reply all
Reply to author
Forward
0 new messages