Opened 7 years ago
Closed 7 years ago
#54448 closed submission (fixed)
new port: sqlitebrowser
Reported by: | RJVB (René Bertin) | Owned by: | mf2k (Frank Schima) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | mf2k (Frank Schima) | |
Port: | sqlitebrowser |
Description
This port provides an up-to-date successor to the old port:sqlitedbrowser
which hasn't been maintained (neither port nor the upstream project).
By default the project builds private copies of all dependencies including QScintilla. I've patched the CMake file to use port:qscintilla2
, and the app PortGroup (combined with the provided Info.plist) to create an app bundle with a shortcut wrapper in $prefix/bin
for those who prefer to invoke the application from the command line.
Attachments (4)
Change History (9)
Changed 7 years ago by RJVB (René Bertin)
Changed 7 years ago by RJVB (René Bertin)
Attachment: | patch-build-for-MP.diff added |
---|
comment:1 Changed 7 years ago by mf2k (Frank Schima)
Changed 7 years ago by mf2k (Frank Schima)
Attachment: | Portfile.2 added |
---|
Here is my modified version. It is simplified with better use of the GitHub portgroup.
comment:2 Changed 7 years ago by mf2k (Frank Schima)
I'm guessing you have a case sensitive file system?
comment:3 Changed 7 years ago by mf2k (Frank Schima)
Here is a working version that:
- Adds your GitHub handle and openmaintainer
- Has a workaround for case insensitive file systems
- Has new checksums (the original ones did not work for me)
Let me know if this is ok with you and I will commit it.
comment:4 Changed 7 years ago by RJVB (René Bertin)
Looks good to me (I don't usually add my github handle but no problem).
The original checksums worked for me with the way I fetched the source, but your implementation is cleaner. Thanks also for the case-insensitive fix, I completely missed the incompatibility.
comment:5 Changed 7 years ago by mf2k (Frank Schima)
Owner: | set to mf2k |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I'm seeing this error:
Also, can we add openmaintainer?