Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add an orgmode mimetype #82

Closed
wants to merge 2 commits into from
Closed

add an orgmode mimetype #82

wants to merge 2 commits into from

Conversation

jkitchin
Copy link
Contributor

This adds a new mimetype for orgmode files (http://orgmode.org).

Copy link
Contributor

@dougwilson dougwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add at least one primary source documenting the link between that file extension and the mime type (https://github.com/jshttp/mime-db#contributing), thanks!

@jkitchin
Copy link
Contributor Author

I have added a link to an email discussing this mimetype, and to the orgmode website. There doesn't seem to be anything listed in IANA (https://www.iana.org/assignments/media-types/media-types.xhtml#text).

Copy link
Contributor

@dougwilson dougwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @jkitchin ! Unfortunately I cannot accept this entry without a link to at least one primary source documenting the link between that file extension and the mime type. That email link is not a primary source, and there is no mention of the extension / mime type of the website you added as a separate link.

I suggest either at least getting the associated between them documented on http://orgmode.org/ or go through the IANA process to reserve the type. If the authors submit an application, that would be a good primary source to add here even before it completes the IANA process.

Thanks!

@dougwilson
Copy link
Contributor

At the very least, that email you linked to ends with:

...which doesn't necessarily mean it's correct. I have no idea myself,
so I'm cc-ing the Org list, perhaps somebody there can provide more
insight.

So if you can link to the response where the orgmode project confirmed the mime association, that would be good enough as well.

@jkitchin
Copy link
Contributor Author

jkitchin commented Sep 5, 2017

The official mimetype of org files is now "The de facto mimetype for org attachments is text/x-org." and is documented at http://orgmode.org/index.html.

@jkitchin
Copy link
Contributor Author

jkitchin commented Sep 6, 2017

Apparently the x- prefix is being deprecated, so the org-mode project now lists the mimetype as "The de facto mimetype for org attachments is text/org".

@josephmturner
Copy link

Hello @dougwilson @jkitchin !

So if you can link to the response where the orgmode project confirmed the mime association, that would be good enough as well.

According to the Org mode website, "The de facto mimetype for Org files is text/org. Org files use the .org extension." Is this good enough for a PR like this one to be merged?

Thank you!

@wesleytodd
Copy link
Member

Hey @josephmturner. I would suggest opening a new PR. If you could also include those references inline into the description. Ideally if you could also get someone to describe what is implied by "defacto" that would be awesome.

@josephmturner
Copy link

@wesleytodd Thanks for the suggestions. I realize now that mime-db already contains an .org => text/x-org mapping.

This mapping is already being ignored in mime-types since it has lower preference than the IANA .org=> application/vnd.lotus-organizer MIME type. So I think that adding an .org => text/org would not meet my need, which is for mime or mime-types to do either of these:

mime.lookup('org') // 'text/org' ('text/x-org' would be okay also)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants