Page MenuHomeFreeBSD

New port: editors/cudatext: Cross-platform text editor, written in Lazarus
AbandonedPublic

Authored by acm on Apr 30 2018, 2:50 PM.
Referenced Files
F81640134: D15240.id.diff
Fri, Apr 19, 8:50 AM
Unknown Object (File)
Mon, Apr 15, 6:38 AM
Unknown Object (File)
Mon, Apr 15, 6:16 AM
Unknown Object (File)
Feb 26 2024, 1:38 AM
Unknown Object (File)
Jan 27 2024, 3:28 AM
Unknown Object (File)
Jan 10 2024, 12:53 PM
Unknown Object (File)
Dec 20 2023, 8:35 AM
Unknown Object (File)
Nov 19 2023, 12:49 AM
Subscribers

Details

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

cudatext/Makefile
45

Remove, set GH_PROJECT=CudaText instead. Isn't there a warning about this exact thing?

linimon retitled this revision from New port: editors/cudatext to New port: editors/cudatext: Cross-platform text editor, written in Lazarus.Aug 2 2018, 10:13 PM
  • Apply mat's suggestions
Makefile
43

Like the warning told you, please remove this line, it is wrong.

45

The flavors are framework generated, you cannot know which will be the default, it would be better to use variables instead of a condition, like:

USE_FPC_gtk2=  cairo
USE_GNOME_gtk2= cairo
USE_FPC+= ${USE_FPC_${FLAVOR}}
USE_GNOME+= ${USE_GNOME_${FLAVOR}}
  • Apply mat's suggestions
acm marked 2 inline comments as done.Aug 6 2018, 6:42 PM
Makefile
33–35

Wrong place in the Makefile. These are flavors helpers. See Chapter 15. Order of Variables in Port Makefiles.

  • Move flavors helpers to right place
acm marked an inline comment as done.Aug 7 2018, 7:43 AM
  • Update to 1.60.0
  • Update dependencies