Closed Bug 1456877 (bmo-libcmark-gfm) Opened 3 years ago Closed 2 years ago

Add a wrapper around libcmark_gfm to Bugzilla

Categories

(bugzilla.mozilla.org :: General, enhancement)

Development
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dylan, Assigned: imadueme)

References

Details

Attachments

(1 file, 1 obsolete file)

45 bytes, text/x-github-pull-request
Details | Review
Adding the CommonMark CPAN module requires an install of libcmark.

There are a number of ways forward on this:

- Figure out how to build it on centos6. We'd need to build it both in the docker containers and in the development VM.

- Find a centos6 rpm from a reputable source  (e.g. needs to receive updates)

- Write an Alien::libcmark CPAN module that will detect a system libcmark or build one. I already did this for libcmark_gfm but we've decided to go forward with the mainline CommonMark implementation. Doing this would also require getting the CommonMark module updated but the maintainer might appreciate that.
  Writing Alien::* modules is very pleasant (mostly just need to adjust the alienfile in https://github.com/dylanwh/Alien-libcmark_gfm)

- Decide we need to update to centos 7 first.
Alias: bmo-libcmark
The end result is to have a set of commands that can be run so that the following command doesn't fail:

cpanm -l test CommonMark

Currently this results in this error:
Running Makefile.PL
libcmark 0.21.0 or higher not found at Makefile.PL line 16.
-> N/A
-> FAIL Configure failed for CommonMark-0.280301. See /home/vagrant/.cpanm/work/1524267352.13714/build.log for details.
Status: NEW → ASSIGNED
Assignee: imadueme → nobody
Status: ASSIGNED → NEW
Assignee: nobody → dylan
Summary: Figure out how to get libcmark installed on centos6 → Add a wrapper around libcmark_gfm to Bugzilla
Attached file PR (obsolete) —
Alias: bmo-libcmark → bmo-libcmark-gfm
Attached file Github PR #598
Attachment #8973865 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Assignee: dylan → imadueme
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.