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 md.obsidian.Obsidian (Obsidian) #1883

Closed
wants to merge 23 commits into from

Conversation

joshua-stone
Copy link

@joshua-stone joshua-stone commented Oct 11, 2020

This PR should fulfill a feature request made on the Obsidian community forum:

https://forum.obsidian.md/t/linux-packaging-ship-as-flatpak-on-flathub/6112

Note: I'm not one of the upstream developers.

@joshua-stone joshua-stone changed the title Md.obsidian.obsidian Add md.obsidian.obsidian (Obsidian) Oct 11, 2020
@nedrichards
Copy link
Member

@nedrichards nedrichards commented Oct 11, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 11, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 11, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 11, 2020

Build 30102 failed

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 11, 2020

I've changed the flatpak to use the tarball instead of the appimage, which hopefully will let the build succeed this time.

@joshua-stone joshua-stone changed the title Add md.obsidian.obsidian (Obsidian) Add md.obsidian.Obsidian (Obsidian) Oct 11, 2020
@barthalion
Copy link
Member

@barthalion barthalion commented Oct 12, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Build 30136 failed

@barthalion
Copy link
Member

@barthalion barthalion commented Oct 12, 2020

Build failed due to missing appdata file:

stat: cannot stat 'builddir/*/share/app-info/xmls/md.obsidian.Obsidian.xml.gz': No such file or directory

md.obsidian.Obsidian.appdata.xml Show resolved Hide resolved
<name>Obsidian</name>
<summary>Markdown-based knowledge base</summary>
<description>
<p>Obsidian is a powerful knowledge base that works on top of a local folder of plain text Markdown files.</p>
Copy link
Member

@barthalion barthalion Oct 12, 2020

Choose a reason for hiding this comment

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

Could the description be expanded upon to include what the homepage says?

md.obsidian.Obsidian.appdata.xml Outdated Show resolved Hide resolved
md.obsidian.Obsidian.yml Outdated Show resolved Hide resolved
@barthalion
Copy link
Member

@barthalion barthalion commented Oct 12, 2020

FYI you can trigger new builds yourself with bot, build md.obsidian.Obsidian.

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 12, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Build 30192 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/29077/md.obsidian.Obsidian.flatpakref

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 12, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 12, 2020

Build 30205 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/29090/md.obsidian.Obsidian.flatpakref

@barthalion
Copy link
Member

@barthalion barthalion commented Oct 13, 2020

My only gripe is short description. When that's improved, it's ready to merge.

@barthalion
Copy link
Member

@barthalion barthalion commented Oct 13, 2020

@lishid, would you like to have write access to the Flathub repository, just in case?

@lishid
Copy link

@lishid lishid commented Oct 13, 2020

@lishid, would you like to have write access to the Flathub repository, just in case?

Sure that would definitely help!

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 13, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Build 30279 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/29164/md.obsidian.Obsidian.flatpakref

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 13, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Build 30281 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/29167/md.obsidian.Obsidian.flatpakref

@joshua-stone
Copy link
Author

@joshua-stone joshua-stone commented Oct 13, 2020

bot, build md.obsidian.Obsidian

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Queued test build for md.obsidian.Obsidian.

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 13, 2020

Build 30282 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/29168/md.obsidian.Obsidian.flatpakref

- cp -r * ${FLATPAK_DEST}
sources:
- type: archive
url: https://github.com/obsidianmd/obsidian-releases/releases/download/v0.9.3/obsidian-0.9.3.tar.gz
Copy link
Member

@barthalion barthalion Oct 14, 2020

Choose a reason for hiding this comment

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

One thing I'd add here is x-data-checker as documented at https://github.com/flathub/flatpak-external-data-checker so Flathub version doesn't fall behind with releases, but that can be fixed after merge.

@barthalion
Copy link
Member

@barthalion barthalion commented Oct 14, 2020

/merge @lishid

@flathubbot
Copy link

@flathubbot flathubbot commented Oct 14, 2020

A repository for this has been created: https://github.com/flathub/md.obsidian.Obsidian

You will receive an invitation to be a collaborator which will grant you write access to the repository above. The invite can be also viewed here.

If you have never maintained an application before, common questions are answered in the app maintenance guide.

Thanks!

@flathubbot flathubbot closed this Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants