Page MenuHomeFreeBSD

Adjust mail/p5-Email-Sender to use p5-Email-Address-XS and p5-Email-Address-UseXS This change removes the existing dependency on p5-Email-Address and adds: * mail/p5-Email-Address-XS * mail/p5-Email-Address-UseXS From https://metacpan.
ClosedPublic

Authored by dvl on Dec 7 2018, 7:57 PM.

Details

Summary

...an.org/pod/Email::Address::UseXS

Email::Address::UseXS - ensure that any code uses Email::Address::XS instead of Email::Address
To use, simply add use Email::Address::UseXS; in your code before anything that tries to load in Email::Address.

Now, some people might want to keep using Email::Address ... I am not sure
how to continue to allow them to do that.

Also, perhaps we should patch the vendor code to add: use Email::Address::UseXS;
That would mean any existing user code would continue to work as expected.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

dvl created this revision.Dec 7 2018, 7:57 PM
ndowens_yahoo.com added inline comments.
mail/p5-Email-Address-UseXS/Makefile
11 ↗(On Diff #51736)

Shouldn’t this be a short description of port, not reason of change

16 ↗(On Diff #51736)

Should go below USES with space per handbook(order of variables)

dvl updated this revision to Diff 51738.Dec 7 2018, 8:47 PM
dvl marked 3 inline comments as done.

Updating D18479: Adjust mail/p5-Email-Sender to use p5-Email-Address-XS and p5-Email-Address-UseXS

This change removes the existing dependency on p5-Email-Address and adds:

  • mail/p5-Email-Address-XS
  • mail/p5-Email-Address-UseXS

From https://metacpan.

mail/p5-Email-Address-UseXS/Makefile
11 ↗(On Diff #51736)

Those words are taken directly from the vendor homepage at https://metacpan.org/pod/Email::Address::UseXS

This revision was not accepted when it landed; it landed in state Needs Review.Jan 4 2019, 7:02 PM
This revision was automatically updated to reflect the committed changes.