-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
base: new-pr
Are you sure you want to change the base?
Conversation
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
org.brlcad.BRL_CAD.metainfo.xml
Outdated
<?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> |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
org.brlcad.BRL_CAD.yaml
Outdated
- name: BRL-CAD-metainfo-desktopfile | ||
buildsystem: simple | ||
build-commands: | ||
- install -Dm644 mged.desktop ${FLATPAK_DEST}/share/applications/${FLATPAK_ID}.mged.desktop |
There was a problem hiding this comment.
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)
There was a problem hiding this comment.
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
org.brlcad.BRL_CAD.yaml
Outdated
|
||
modules: | ||
- name: BRL-CAD | ||
buildsystem: cmake |
There was a problem hiding this comment.
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
?
Started test build 55159 |
Build 55159 failed |
org.brlcad.BRL_CAD.yaml
Outdated
cflags: -Wno-error | ||
cxxflags: -Wno-error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
org.brlcad.BRL_CAD.yaml
Outdated
type: git | ||
tag-pattern: ^rel-([\d-]+)$ | ||
|
||
# .desktop files is removed from /misc/debian after install, so manually install them. |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this 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.
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 55244 |
Build 55244 failed |
oof, the website ssl cert expired |
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Build 55437 failed |
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 55499 |
Build 55499 failed |
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 56742 |
Build 56742 successful
|
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 57679 |
Build 57679 successful
|
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 60342 |
Build 60342 successful
|
org.brlcad.BRL_CAD.yaml
Outdated
@@ -0,0 +1,67 @@ | |||
app-id: org.brlcad.BRL_CAD | |||
runtime: org.kde.Sdk | |||
runtime-version: 6.5 |
There was a problem hiding this comment.
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
anything else to mark this as not |
bot, build org.brlcad.BRL_CAD |
Queued test build for org.brlcad.BRL_CAD. |
Started test build 112243 |
Build 112243 failed |
Please confirm your submission meets all the criteria