Opened 14 years ago
Closed 14 years ago
#633 closed enhancement (fixed)
Include platform in notification mail
| Reported by: | Felix Schwarz <felix.schwarz@…> | Owned by: | hodgestar |
|---|---|---|---|
| Priority: | major | Milestone: | 0.6 |
| Component: | Trac plugin | Version: | dev |
| Keywords: | Cc: | ||
| Operating System: | Linux |
Description
We have a couple of build slaves. Build slaves have a not so meaningful name as they equal the computer name. The most important information is in the platform name. Therefore I'd like to see the platform name to be displayed in notification emails.
Attachments (1)
Change History (6)
Changed 14 years ago by hodgestar
comment:1 Changed 14 years ago by hodgestar
I've attached a patch that adds the platform to the information to the build notification email template. The existing test suite passes but the new functionality isn't really tested because adding a platform to the current tests would require some reworking.
Feedback, testing and/or updated patches welcomed.
comment:2 Changed 14 years ago by hodgestar
- Cc hodgestar added
comment:3 Changed 14 years ago by osimons
- Milestone changed from 0.6.1 to 0.6
I'm OK with adding this + #541 now. They just provide more context to the information being displayed.
comment:4 Changed 14 years ago by osimons
- Cc hodgestar removed
- Owner set to hodgestar
Part of [953]. Will be backported to 0.6 shortly.
comment:5 Changed 14 years ago by osimons
- Resolution set to fixed
- Status changed from new to closed
... and closing.
Patch to add platform to build notification emails.