Created attachment 219348 [details]
New port: www/dooble web browser.
Dooble, the weather bug browser. Minimal, cute, and unusually stable.
- A Web browser!
- Basic themes.
- Cookie crusher.
- Custom search engines.
- Custom style sheets.
- Domain restrictions.
- Floating digital clock.
- FreeBSD, Linux, Mac, Windows.
- Gopher support.
- Multiple private instances, including downloads.
- Original implementations of AES-256, Threefish-256.
- Qt-only dependency.
- Windows portable.
Looks good a first glance but I don't think you should override ports framework regarding CFLAGS such as optimization at least not by default and removing flags such as -std=c++11 may cause compilation issues on some platforms. Is it really a good idea to strip pretty much everything upstream sets?
(In reply to daniel.engberg.lists from comment #1)
OK, I will remove optimization flags which should get from ports system and flags that fail compiling.
Created attachment 219349 [details]
Only remove optimization CXXFLAGS and flags that fail compiling.
Created attachment 219389 [details]
I've adjust the patch a little bit to make `portlint -abct` happy, please check it. Thanks!
you might want to assign QT5_VERSION since changes to QT could cause build failures. I actually attempted to build this (by setting up a test port using your diff) but using an older QT (5.13.0) and there were some build errors, apparently due to name changes in the QT web stuff.
That being the case, 3rd party dependencies like QT can become moving targets and cause build failures. I've seen the "moving target" problem with other massive libraries, and setting QT5_VERSION might give you a heads up on this.
The current (2020 Q4) value (from Uses/qt.mk) is "5.15.0", FYI
(In reply to my own from comment #5)
actually may be better to check, rather than assign, QT5_VERSION since the ports make environment doesn't seem to do the version checks. it does use a '?=' to assign it to the current version, which I would think implies that you can assign it to something else. But since it's not obvious (to me anyway) where this is being checked, if at all, it's probably better to check for it and compare the version to whatever you know actually builds, and warn (or fail or apply additional patches) for anything else.
Created attachment 219471 [details]
Add QT5 minor version check
Add QT5 minor version check.
Created attachment 219526 [details]
Fix QT5 minor version check