|Re: [mail-dev] [External] : Correct place to submit PR for JavaMail/JakartaMail|
Hi, On 9/16/21 5:59 AM, Hiren Joshi wrote:
Hi, I have a patch to submit for JavaMail/JakartaMail.The JavaMail home page on oracle.com <https://www.oracle.com/java/technologies/javamail.html> pointed me to a read-only archived JavaMail github repository <https://urldefense.com/v3/__https://github.com/javaee/javamail__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCqICZynQ$> which pointed me to the JakartaMail eclipse github repository <https://urldefense.com/v3/__https://github.com/eclipse-ee4j/mail__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCZIzBJ9o$>.So I have opened a PR there - https://github.com/eclipse-ee4j/mail/pull/580 <https://urldefense.com/v3/__https://github.com/eclipse-ee4j/mail/pull/580__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCfutjQqY$>
for bugs/changes in the code covered by the Jakarta Mail spec - jakarta.mail package namespace, that is the right place.
However the readme file <https://urldefense.com/v3/__https://github.com/eclipse-ee4j/mail/blob/master/README.md__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCIzbdjbw$> says the implementation is moved to Eclipse Angus <https://urldefense.com/v3/__https://github.com/eclipse-ee4j/angus-mail__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCBAwYnaE$>, but I don't see the file I have patched there.
Implementation is everything outside of the jakarta.mail packages; ie if the bug is in com.sun.mail package then it belongs to the implementation, in this case to the angus project
Please advise. Thanks, Hiren _______________________________________________ mail-dev mailing list mail-dev@xxxxxxxxxxx To unsubscribe from this list, visit https://urldefense.com/v3/__https://www.eclipse.org/mailman/listinfo/mail-dev__;!!ACWV5N9M2RV99hQ!Za8QG5eQfolMjeYktZ5bIsoZzcNZBTkEG4C5-Ew9SCJbR-_s_K7I24HysSkCEHli0t0$
Back to the top