Update MACOSX_DEPLOYMENT_TARGET to 10.13

[ejguan]

🚀 Feature

Motivation

TorchData is trying to enable AWSSDK to binary by default. However, the AWSSDK requires at least 10.13 for MacOS. Since TorchText is currently using MACOSX_DEPLOYMENT_TARGET=10.9, TorchData has to postpone enable AWSSDK in MacOS by default.

Pitch

Could we update the minimum version of MACOSX_DEPLOYMENT_TARGET in TorchText?

@atalman recommended me to post an issue to see if this is an option for TorchText.

Alternatives

Additional context

[mthrok]

I am not macOS expert and I do not really know what it does, but from what I have been told and have done in audio, there should be no issue removing MACOSX_DEPLOYMENT_TARGET from TorchText. (or updating it)

[ejguan]

Thanks @mthrok for your input! That would be perfect for TorchData.

@parmeet wdyt?

Note: We don't necessarily make it into this coming release. I can disable C++ extension for TorchData on MacOS for the coming release.

[parmeet]

Yes, I agree with what @mthrok said. I don't think this is an issue..Will make PR similar to Audio to remove it from TorchText.

[ejguan]

@parmeet Do you think it's going to be shipped before release?

[ejguan]

Closing as the PR has been merged. Thank you @parmeet