Opened 7 years ago

Closed 6 years ago

#3343 closed defect (fixed)

v.label.sa broken and duplicated in trunk and addons

Reported by: wenzeslaus Owned by: grass-dev@…
Priority: trivial Milestone: 7.4.0
Component: Addons Version: svn-trunk
Keywords: v.label.sa, FTLIB, FreeType, compilation Cc:
CPU: Unspecified Platform: Unspecified

Description

v.label.sa code is both in trunk and in addons repository. The versions are the same except for few characters. The compilation is disabled in trunk and it is broken in addons with:

labels.c: In function ‘labels_init’:
labels.c:88:5: error: implicit declaration of function ‘find_font_from_freetypecap’ [-Werror=implicit-function-declaration]
labels.c:88:14: warning: assignment makes pointer from integer without a cast [enabled by default]
labels.c:100:5: error: implicit declaration of function ‘free_fontcap’ [-Werror=implicit-function-declaration]

In G6 the compilation is conditional:

REQUIRESFT = v.label.sa

ifneq ($(strip $(FTLIB)),)
    SUBDIRS += $(REQUIRESFT)
endif

and the v.label.sa code in G6 contains the missing function find_font_from_freetypecap().

The free_fontcap function is in lib/driver/driverlib.h in G7.

I suggest to remove the trunk version. Fixing the code would be nice of course.

Change History (3)

in reply to:  description comment:1 by mmetz, 7 years ago

Replying to wenzeslaus:

v.label.sa code is both in trunk and in addons repository. The versions are the same except for few characters. The compilation is disabled in trunk and it is broken in addons with:

labels.c: In function ‘labels_init’:
labels.c:88:5: error: implicit declaration of function ‘find_font_from_freetypecap’ [-Werror=implicit-function-declaration]
labels.c:88:14: warning: assignment makes pointer from integer without a cast [enabled by default]
labels.c:100:5: error: implicit declaration of function ‘free_fontcap’ [-Werror=implicit-function-declaration]

With r71061, the compile errors in v.label.sa are fixed and the versions in trunk and addons have been sync'ed.

comment:2 by neteler, 6 years ago

Re-enabled module in r71684.

comment:3 by neteler, 6 years ago

Resolution: fixed
Status: newclosed

Closing.

Note: See TracTickets for help on using tickets.