History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: QB-650
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Won't Fix
Priority: Major Major
Assignee: Robin Shen
Reporter: Roger Perkins
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
QuickBuild

Ability in Build Condition to also compare based on name of the committer

Created: 28/Sep/10 09:57 AM   Updated: 28/Sep/10 12:49 PM
Component/s: None
Affects Version/s: 3.0.7
Fix Version/s: None

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown


 Description  « Hide
We have assets that are checked back into our perforce repository as a result of some builds. All the commits are by the same "automated" user account and it would be ideal to be able to filter out commits made by this user so they don't trigger needless CI when the build machine next checks for changes. Right now I can only easily check for changes to excluse based on file path (which could get overly complex for this scenario).

For the occasions where you might want slight variations on the default options provided, it would also be really nice to have some way to see what the equivelant script for the default options would be so that it could be copied and modified into a custom script by the user.

 All   Comments   Work Log   Change History      Sort Order:
Roger Perkins [28/Sep/10 09:58 AM]
Ok... missed the priority... not exactly "major"... I can probably fumble around and work out the script to do this, but it might take a while to get right and gets the bugs out.

Robin Shen [28/Sep/10 12:24 PM]
I agree to provide default script in some form to help writing custom scripts. I will create another improvement request and close this ticket. For this specific case, below script should work:
for (repository: configuration.getReferencedRepositories()) {
  boolean developerCommitsFound = false;
  for (change: repository.getChanges()) {
    if (!change.getUser().equals("automated"))
      developerCommitsFound = true;
  }
  if (developerCommitsFound)
    return true;
}
return false;