Page MenuHomeFreeBSD

databases/sqlitestudio: Unbroke the build by including QVariant where it was missing
ClosedPublic

Authored by yuri on Nov 23 2017, 2:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 11:56 PM
Unknown Object (File)
Sun, Dec 1, 5:44 AM
Unknown Object (File)
Fri, Nov 29, 7:52 AM
Unknown Object (File)
Nov 25 2024, 9:57 PM
Unknown Object (File)
Nov 25 2024, 3:59 PM
Unknown Object (File)
Nov 23 2024, 1:17 AM
Unknown Object (File)
Nov 19 2024, 11:53 PM
Unknown Object (File)
Nov 19 2024, 6:03 AM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 12959
Build 13217: arc lint + arc unit

Event Timeline

databases/sqlitestudio/Makefile
4

^Why change this, it also requires the DISTNAME, and the port ist named sqlitebrowser :)

yuri marked an inline comment as done.Nov 23 2017, 5:43 AM
yuri added inline comments.
databases/sqlitestudio/Makefile
4

They call it SQLiteStudio here: https://sqlitestudio.pl

databases/sqlitestudio/Makefile
4

I don't think that is worth the change -- if the distfile were called that too, yes.

yuri marked an inline comment as done.

Restored the package name.

yuri retitled this revision from databases/sqlitestudio: Unbroke the build; Changed the package name to mixed case to reflect how it calls itself to databases/sqlitestudio: Unbroke the build by including QVariant where it was missing.Nov 23 2017, 6:16 AM
yuri marked 2 inline comments as done.Nov 24 2017, 7:19 AM
mat requested changes to this revision.Nov 24 2017, 6:30 PM
mat added inline comments.
databases/sqlitestudio/Makefile
4–6

Do not change PORTVERSION if the version does not change.

This revision now requires changes to proceed.Nov 24 2017, 6:30 PM

Brought back PORTVERSION.

yuri marked an inline comment as done.Nov 24 2017, 9:02 PM

@mat

PORTVERSION has been changed back. Is there anything else? Otherwise, could you please approve it?

Thanks!

This revision is now accepted and ready to land.Nov 27 2017, 1:44 PM
This revision was automatically updated to reflect the committed changes.