Details
- Reviewers
DataDrake JoshStrobl - Group Reviewers
Triage Team - Maniphest Tasks
- T9287: Update msmtp to latest release 1.8.12
- Commits
- R2110:ea37fcce40db: msmtp: Update to latest release 1.8.15
Build, installed and tested on one device. Fixed the TLS error reported in T9287
Diff Detail
- Repository
- R2110 msmtp
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
@palto42 could you please add Fixes T9287 in the Summary? So that this patch gets automatically associated to the task you're solving.
@livingsilver94 Hope the format is ok now, I'm new to Arcanist and was not sure about the formatting of the commit message.
Should be OK now! There's just a small typo you can fix ("lates"), and you didn't provide a changelog. Given how many versions have been skipped, I think it's safe to just add Changelog available [here](https://marlam.de/msmtp/news/) from X to Y. in the Summary.
There's not an enforced format by the way, only some common templates.
See inline comments.
| package.yml | ||
|---|---|---|
| 12 | We don't do multi-line descriptions unless it's really needed. I think the summary text is fine in this case, but you are welcome to write a 2-3 sentence description as long as it's concise and readable. | |
| package.yml | ||
|---|---|---|
| 12 | Hope this shorter description is ok. | |
| package.yml | ||
|---|---|---|
| 12 | It's better but still doesn't need to be multi-line. | |
@JoshStrobl Sorry to say, but I'm getting a bit frustrated to wait that long for a quite simple update to get merged. Just wanted to help and contribute to the Solus project, but I get the feeling that my help is not much appreciated.
It also makes me worry to see packages in the repository without any updates since years, which could cause potential security risks (not in this case, but maybe for other packages).
Will merge this when I have the time.
You know...seeing as I am a volunteer. Nice to know you value my time too.