#1235 Add Lua packaging guidelines
Closed by james. Opened by fed500.
Unknown source master

Download 1235.patch
no initial comment

lua(abi) should be required automatically when packages install files to a specific location.

1 new commit added

  • Requires lua(abi) no longer needed

Thanks. Removed that line.

Metadata Update from @tibbs:
- Pull-request tagged with: meeting

Not sure we need this paragraph. Other language specific guidelines don't have this either. But having it probably doesn't hurt ...

This doesn't render correctly. Code blocks that are to be included verbatim need to be surrounded by .... (four dots).

Alternatively, if you want syntax highlighting, the format to use is

[source,<language>]
----
<contents>
----

This is missing the second plus character, should be `+lua-modulename+.

We don't keep historical records in the Packaging Guidelines. There's no need to say "only supported on Fedora >= X" if all current branches already support it.

Same here, needs to be four dots, not three.

And again, four dots, not three.

EPEL specific guidelines are maintained separately:
https://docs.fedoraproject.org/en-US/epel/epel-packaging/

But I'm not sure it's worth documenting a small thing like that separately, so it's probably fine to keep it here.

I left some comments, mostly wrt/ AsciiDoc syntax / formatting problems. Sorry that this is taking so long ...

Fixed the AsciiDoc syntax problems, rebased on top of master: https://pagure.io/packaging-committee/pull-request/1298

Note that decathorpe made the asciidoc fixes, and opened a new PR (which is getting merged): https://pagure.io/packaging-committee/pull-request/1298

Pull-Request has been closed by james

Metadata