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

Compile md4c into QOwnNotes #3080

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Compile md4c into QOwnNotes #3080

merged 2 commits into from
Aug 8, 2024

Conversation

Waqar144
Copy link
Contributor

@Waqar144 Waqar144 commented Aug 8, 2024

Ideally QOwnNotes should use the system md4c but its not possible atm.

Ideally QOwnNotes should use the system md4c but its not possible atm.
DEV_MODE currently enables -Werror on linux
@pbek
Copy link
Owner

pbek commented Aug 8, 2024

Ideally QOwnNotes should use the system md4c but its not possible atm.

So what is the PR doing, solving or improving? Is it for #3076? Don't we need https://github.com/qownnotes/md4c?

@pbek
Copy link
Owner

pbek commented Aug 8, 2024

Ah, really for #3076, thank you very much!

@pbek pbek merged commit 77eb501 into pbek:main Aug 8, 2024
25 of 30 checks passed
@pbek pbek mentioned this pull request Aug 8, 2024
6 tasks
@Waqar144
Copy link
Contributor Author

Waqar144 commented Aug 8, 2024

Ideally QOwnNotes should use the system md4c but its not possible atm.

So what is the PR doing, solving or improving? Is it for #3076? Don't we need https://github.com/qownnotes/md4c?

Just treating md4c as part of QOwnNotes source instead of linking it into QOwnNotes. This is what we do in the qmake file.

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

Successfully merging this pull request may close these issues.

2 participants