luasocket icon indicating copy to clipboard operation
luasocket copied to clipboard

Please ship `mbox.lua` in rock

Open rrthomas opened this issue 4 years ago • 6 comments

This useful module is shipped by Debian, for example, but not in the LuaRock. It seems to be the only Lua module in the sources that is not shipped.

rrthomas avatar Jan 10 '21 12:01 rrthomas

what does the module do? does it conform to some protocol?

Tieske avatar Mar 23 '22 16:03 Tieske

(asking since it is not documented)

Tieske avatar Mar 23 '22 16:03 Tieske

It parses mbox files. I wrote my own module to do this before realising there was already one in luasocket.

rrthomas avatar Mar 24 '22 08:03 rrthomas

Pardon me if I'm being dense, but why is a networking / socket library a good home for a file format parser? That sounds like something that should have a different home. Parsing an over the while protocol is one thing, parsing an on disk file format seems to me to be a category error. What utility does Debian get out of it being in this package?

alerque avatar Mar 24 '22 08:03 alerque

I don't know why it was added (it was in the initial git revision in 2002), but the mbox.lua module has been maintained ever since. mbox is, after all, a file format closely associated with networking.

If the maintainers would rather split it out into a new library, that would be fine too.

The main point is simply that there is an mbox parser, and it would be nice to be able to install and use it via the standard luarocks route.

rrthomas avatar Mar 24 '22 09:03 rrthomas

Having looked this over I am still of the opinion that it should be broken out into a different project ... but in the mean time given the way we are using it for tests and the example code that provides I think it's entirely reasonable to ship it with the rock until such a time as it can be reasonably installed as a dependency from Luarocks.

alerque avatar Mar 25 '22 10:03 alerque

Thanks for doing this!

rrthomas avatar Nov 10 '23 22:11 rrthomas