Skip to content
Permalink
Browse files

Merge pull request #49 from HexTank/master

Fix for JENKINS-11600, email-ext now queries getTimestamp and getCommite...
  • Loading branch information...
rpetti committed Jul 17, 2014
2 parents 2da2bf9 + fda9b70 commit e49e9cb874d9536210aa9c90119c96ce50a1ad82
Showing with 11 additions and 0 deletions.
  1. +11 −0 src/main/java/hudson/plugins/perforce/PerforceChangeLogEntry.java
@@ -65,9 +65,14 @@ public String getChangeNumber() {
}

//used for email-ext
@Deprecated
public String getRevision() {
return getChangeNumber();
}
//used for email-ext
public String getCommitId() {
return getChangeNumber();
}

@Exported
public String getChangeTime() {
@@ -76,9 +81,15 @@ public String getChangeTime() {
}

//used for email-ext
@Deprecated
public String getDate() {
return getChangeTime();
}
//used for email-ext
public long getTimestamp() {
return getChange().getDate().getTime();
}


/**
* {@inheritDoc}

5 comments on commit e49e9cb

@oleg-nenashev

This comment has been minimized.

Copy link
Member

replied Oct 1, 2014

@rpetti , are you going to release the plugin soon? The current SNAPSHOT version works well on our dev. servers

@rpetti

This comment has been minimized.

Copy link
Member Author

replied Oct 1, 2014

I would like to, but I can't build it and I don't have any time to set up a new development environment and track down all the deprecated dependencies it still needs.

@oleg-nenashev

This comment has been minimized.

Copy link
Member

replied Oct 1, 2014

Would you mind if I release it with 1.3.28 or 1.4.0 version?

@rpetti

This comment has been minimized.

Copy link
Member Author

replied Oct 1, 2014

Sure. If you think there have been enough enhancements and architectural changes to warrant a jump to 1.4.0, then go for it.

@oleg-nenashev

This comment has been minimized.

Copy link
Member

replied Oct 2, 2014

Hmm...the release failed due to the authorization issue during the upload to http://maven.jenkins-ci.org. Credentials/Maven settings seems to be OK. I'll check with @rtyler if everything is OK with the infrastructure

Please sign in to comment.
You can’t perform that action at this time.