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

Added flatpak build manifests with split dependencies #1768

Open
wants to merge 1 commit into
base: v0.6
Choose a base branch
from

Conversation

kdcis
Copy link
Contributor

@kdcis kdcis commented May 27, 2021

  • Added Flatpak build manifests under packages/flatpak/ directory

  • Added Flatpak build instructions under docs/ directory

Copy link
Member

@PeterSurda PeterSurda left a comment

Choose a reason for hiding this comment

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

Please update documentation

docs/Flatpak_Build.md Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
docs/Flatpak_Build.md Outdated Show resolved Hide resolved
@g1itch
Copy link
Collaborator

g1itch commented Jun 23, 2021

Also, I don't get a point of documenting the maintenance tasks in such way. Why don't you just make a script buildscripts/flatpak.sh covering the steps under the ##Export header?

There supposed to be a builder on https://git.bitmessage.org/Bitmessage for building and uploading the bundle, which will be used as github check for a new code, like buildbot/PyBitmessage-wine32 and buildbot/dpkg.

@PeterSurda
Copy link
Member

@g1itch I'm working on the buildbot integration.

Copy link
Member

@PeterSurda PeterSurda left a comment

Choose a reason for hiding this comment

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

Clean up, rebase and then I'll look at it again.

@kdcis kdcis force-pushed the v0.6-flatpak branch 4 times, most recently from 07316eb to a9d9356 Compare August 4, 2021 12:29
@kdcis kdcis requested a review from PeterSurda August 4, 2021 16:42
# PyBitmessage Linux flatpak instructions

## Requirements
First make sure you have `flatpak` and `flatpak-builder` installed along with these dependencies (Ubuntu 20.4) `git xz-utils elfutils librsvg2-common` needs to be installed.
Copy link
Member

Choose a reason for hiding this comment

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

line too long

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As it is a readme file, I don't think there would be any pylint line too long warning will happen.
Please confirm if still I need to break it.

Copy link
Member

Choose a reason for hiding this comment

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

I don't have a strong opinion so we can probably just leave it as it is.

@kdcis kdcis force-pushed the v0.6-flatpak branch 7 times, most recently from 48ee56a to eaadcf0 Compare August 26, 2021 15:07
Copy link
Member

@PeterSurda PeterSurda left a comment

Choose a reason for hiding this comment

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

I think we can merge it, the buildbot integration can be deployed later.

# PyBitmessage Linux flatpak instructions

## Requirements
First make sure you have `flatpak` and `flatpak-builder` installed along with these dependencies (Ubuntu 20.4) `git xz-utils elfutils librsvg2-common` needs to be installed.
Copy link
Member

Choose a reason for hiding this comment

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

I don't have a strong opinion so we can probably just leave it as it is.

@kdcis kdcis force-pushed the v0.6-flatpak branch 3 times, most recently from 26216cb to 1753c97 Compare September 2, 2021 07:34
@kdcis kdcis force-pushed the v0.6-flatpak branch 3 times, most recently from 9b95345 to bfa611b Compare September 8, 2021 08:19
@kdcis kdcis force-pushed the v0.6-flatpak branch 7 times, most recently from 57b05f9 to e607cda Compare October 5, 2021 06:53
@kdcis kdcis force-pushed the v0.6-flatpak branch 5 times, most recently from 0036dcd to 2ce0698 Compare October 12, 2021 10:14
@kdcis kdcis force-pushed the v0.6-flatpak branch 4 times, most recently from d0f73d0 to f4edee1 Compare October 14, 2021 09:09
@kdcis kdcis force-pushed the v0.6-flatpak branch 3 times, most recently from 73d8238 to d42f807 Compare October 29, 2021 07:01
@kdcis kdcis force-pushed the v0.6-flatpak branch 2 times, most recently from c1d6e7d to 0a718dc Compare November 3, 2021 07:22
@kdcis kdcis force-pushed the v0.6-flatpak branch 5 times, most recently from 65ef4bb to 475dca3 Compare November 15, 2021 16:21
Removed extra lines from build sequence

Added Dockerfile for Flatpak repo-server & Build instructions under docs/ dir

Modified Dockerfile & Flatpak build doc

updated docs

fixed code quality issues

fixed code quality, added doc string

fixed branch & added build commands

run setuptools

fixed code quality

refactor pip install

updated flatpak build doc & added link to index.rst

Removed extra lines from build sequence

Added Dockerfile for Flatpak repo-server & Build instructions under docs/ dir

Modified Dockerfile & Flatpak build doc

updated docs

fixed code quality issues

fixed code quality, added doc string

fixed branch & added build commands

run setuptools

fixed code quality

refactor pip install

updated flatpak build doc & added link to index.rst

fixed docs

rebased

removed unnecessary files

removed .gitmodules: should be part of build process
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants