Stefan, can you grant me write access to the git repository? I've checked
these pull requests twice now and to me they look just fine.

2015-11-01 12:08 GMT+01:00 Dominik Psenner <dpsen...@gmail.com>:

> At first glance this looks good. But i will need some time to get my hands
> on a computer.
> On 1 Nov 2015 11:48 a.m., "Albin Sunnanbo (JIRA)" <j...@apache.org> wrote:
>
>>
>>     [
>> https://issues.apache.org/jira/browse/LOG4NET-473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14984343#comment-14984343
>> ]
>>
>> Albin Sunnanbo commented on LOG4NET-473:
>> ----------------------------------------
>>
>> Hi again
>> It looks like my SVN skills have been decaying. Sorry for that.
>> I found the GIT-version of the source and supplied a fixed version of the
>> above patch as a pull request in
>> https://github.com/apache/log4net/pull/19
>> I also created a basic test for the SmtpPickupDirAppender in
>> https://github.com/apache/log4net/pull/17
>>
>> Additionally I created a pull request for generating a Date header in
>> https://github.com/apache/log4net/pull/18
>> I prefer separate pull requests for each feature in GIT, but if you
>> prefer the contribution as a single pull request or an SVN path I can fix
>> that.
>>
>> > Option for file extension in SmtpPickupDirAppender
>> > --------------------------------------------------
>> >
>> >                 Key: LOG4NET-473
>> >                 URL: https://issues.apache.org/jira/browse/LOG4NET-473
>> >             Project: Log4net
>> >          Issue Type: New Feature
>> >    Affects Versions: 1.2.13
>> >            Reporter: Albin Sunnanbo
>> >         Attachments: SmtpPickupDirAppender-fileextension.patch
>> >
>> >
>> > Especially on Windows file extensions are very handy, if you give the
>> output files the `eml` extension you can just double click the file to open
>> the mail in outlook.
>> > I have added a `FileExtension` option to the `SmtpPickupDirAppender`
>> that can be used to specify a file extension for the generated files.
>> > Although a default value of `eml` would be nice I have preserved
>> backwards compatibility of no file extension by default.
>>
>>
>>
>> --
>> This message was sent by Atlassian JIRA
>> (v6.3.4#6332)
>>
>


-- 
Dominik Psenner

Reply via email to