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

[WIP] Add org.brlcad.BRL_CAD #4363

Open
wants to merge 19 commits into
base: new-pr
Choose a base branch
from

Conversation

kevinsmia1939
Copy link

Please confirm your submission meets all the criteria

  • I have read the App Requirements and App Maintenance pages.
  • My pull request follows the instructions at App Submission.
  • I am using only the minimal set of permissions. (If not, please explain each non-standard permission.)
  • All assets referenced in the manifest are redistributable by any party. If not, the unredistributable parts are using an extra-data source type.
  • I am an upstream contributor to the project. If not, I contacted upstream developers about submitting their software to Flathub. Link:Package BRL-CAD as Flatpak for Linux distros BRL-CAD/brlcad#88
  • I own the domain used in the application ID or the domain has a policy for delegating subdomains (e.g. GitHub, SourceForge).
  • Any additional patches or files have been submitted to the upstream projects concerned. (If not, explain why.) Metainfo submitted Add metainfo for BRL-CAD BRL-CAD/brlcad#87

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

org.brlcad.BRL_CAD.yaml Outdated Show resolved Hide resolved
<?xml version="1.0" encoding="UTF-8"?>
<component type="desktop">
<id>org.brlcad.BRL_CAD</id>
<launchable type="desktop-id">org.brlcad.BRL_CAD.mged.desktop</launchable>
Copy link
Contributor

Choose a reason for hiding this comment

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

this should match the app-id.

Copy link
Author

Choose a reason for hiding this comment

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

BRL-CAD is a collection of smaller software, Mged, archer, rtwizard. With mged is where things usually start.
So I'm not sure what to do here. Because org.brlcad.BRL_CAD.desktop itself does not exist. Only org.brlcad.BRL_CAD.{mged,archer,rtwizard,db}.desktop exist.

Copy link
Contributor

Choose a reason for hiding this comment

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

but you chose one command, didn't you?

Copy link
Author

Choose a reason for hiding this comment

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

yeah, i guess i will go with mged then

- name: BRL-CAD-metainfo-desktopfile
buildsystem: simple
build-commands:
- install -Dm644 mged.desktop ${FLATPAK_DEST}/share/applications/${FLATPAK_ID}.mged.desktop
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be just ${FLATPAK_ID}.desktop. same for the icon. (based on what command is)

Copy link
Author

Choose a reason for hiding this comment

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

ok, I will use mged.desktop as ${FLATPAK_ID}.desktop


modules:
- name: BRL-CAD
buildsystem: cmake
Copy link
Contributor

Choose a reason for hiding this comment

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

can it build with cmake-ninja ?

@flathubbot
Copy link

Started test build 55159

@flathubbot
Copy link

Build 55159 failed

Comment on lines 21 to 22
cflags: -Wno-error
cxxflags: -Wno-error
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
cflags: -Wno-error
cxxflags: -Wno-error
cflags: -Wno-error=stringop-overflow -Wno-error=format-truncation -Wno-error=inline
-Wno-error=format-overflow -Wno-error=dangling-pointer -Wno-error=array-bounds -Wno-error=nonnull -Wno-error=maybe-uninitialized
cxxflags: -Wno-error=stringop-overflow -Wno-error=format-truncation -Wno-error=inline
-Wno-error=format-overflow -Wno-error=dangling-pointer -Wno-error=array-bounds -Wno-error=nonnull -Wno-error=maybe-uninitialized

my bad this actually doesn't work.

upstream should fix it though.

type: git
tag-pattern: ^rel-([\d-]+)$

# .desktop files is removed from /misc/debian after install, so manually install them.
Copy link
Contributor

Choose a reason for hiding this comment

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

no they are not.

$ find . -name \*.desktop
./misc/debian/archer.desktop
./misc/debian/brlcad-db.desktop
./misc/debian/brlcad-doc-mged.desktop
./misc/debian/brlcad-doc.desktop
./misc/debian/mged.desktop
./misc/debian/rtwizard.desktop

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, I was looking at the build dir not the top source dir. I change it now.

Copy link
Contributor

@hfiguiere hfiguiere left a comment

Choose a reason for hiding this comment

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

Icon and images from upstream should be used instead.

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 55244

@flathubbot
Copy link

Build 55244 failed

@kevinsmia1939
Copy link
Author

oof, the website ssl cert expired

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

org.brlcad.BRL_CAD.yaml Outdated Show resolved Hide resolved
@flathubbot
Copy link

Build 55437 failed

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 55499

@flathubbot
Copy link

Build 55499 failed

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 56742

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/39374/org.brlcad.BRL_CAD.flatpakref

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 57679

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/40303/org.brlcad.BRL_CAD.flatpakref

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 60342

@flathubbot
Copy link

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

flatpak install --user https://dl.flathub.org/build-repo/42975/org.brlcad.BRL_CAD.flatpakref

@@ -0,0 +1,67 @@
app-id: org.brlcad.BRL_CAD
runtime: org.kde.Sdk
runtime-version: 6.5
Copy link
Contributor

Choose a reason for hiding this comment

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

should update to 6.6

@hfiguiere
Copy link
Contributor

anything else to mark this as not [WIP]

@kevinsmia1939
Copy link
Author

bot, build org.brlcad.BRL_CAD

@flathubbot
Copy link

Queued test build for org.brlcad.BRL_CAD.

@flathubbot
Copy link

Started test build 112243

@flathubbot
Copy link

Build 112243 failed

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.

3 participants