Page MenuHomeSolus

Inclusion of subsurface
ClosedPublic

Authored by flyingP0tat0 on Dec 23 2018, 10:18 PM.
Tags
None
Referenced Files
F11009823: D4803.diff
Wed, Aug 2, 8:05 PM
F10988747: D4803.diff
Mon, Jul 24, 12:25 PM
F10945422: D4803.id11750.diff
Sat, Jul 15, 4:05 AM
F10913014: D4803.id11875.diff
Jul 7 2023, 8:16 PM
F10895879: D4803.id.diff
Jul 1 2023, 8:13 PM
F10870180: D4803.id11830.diff
Jun 18 2023, 8:38 AM
F10870153: D4803.id.diff
Jun 18 2023, 8:35 AM
F10852837: D4803.id11799.diff
Jun 13 2023, 5:37 AM

Details

Summary

Depends on D4802

Subsurface was requested per T6796.

Test Plan

Used Subsurface to create and modify dives, imported data from dive computer via USB.

Diff Detail

Repository
R4628 subsurface
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Added qt5-location-googlemaps dependency and needed patch for the dependency to work.

DataDrake added a subscriber: DataDrake.

Needs a rebuild without the googlemaps stuff. Let's wait until upstream officially supports it.

This revision now requires changes to proceed.Dec 26 2018, 2:03 PM

Removed qt5-location-googlemaps dependency

JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
7–8

Firstly, we use SPDX 3.0 identifiers, as stated in our documentation. Secondly this can be on the same line, no reason for it to be a list.

10

No reason for the description to not also be the summary. Divelog program is too succinct.

22

Not necessary. This is a dependency on qt5-connectivity-devel, qt5-svg-devel, etc.

31–33

Can try just using -B build instead.

35–36

Can just use -C build instead of cd'ing first.

38–39

Can just use -C build instead of cd'ing first.

pspec_x86_64.xml
24

Is there a specific reason we need to enable debug for this?

This revision now requires changes to proceed.Dec 26 2018, 7:52 PM
This revision is now accepted and ready to land.Dec 27 2018, 7:32 PM
This revision was automatically updated to reflect the committed changes.