Opened 10 years ago

Closed 8 years ago

#867 closed defect (wontfix)

WinGrass64-compiling: `libintl_printf' is an unrecognized format function type'

Reported by: hellik Owned by: grass-dev@…
Priority: minor Milestone: 6.4.0
Component: Compiling Version: svn-releasebranch64
Keywords: wingrass Cc: Windows, compiling
CPU: x86-32 Platform: MSWindows Vista

Description

i've compiled the Grass64-releasebranch in the osgeo4w-stack on WinVista32 (--nls enabled).

unfortunately I wasn't all the building time in front of my laptop and couldn't catch all the messages, but I've seen in a few-modules (raster and vector etc.; i.e. nviz, v.what, v.out.pov, v.support, v.transform, v.to.db ) following warnings:

`libintl_printf' is an unrecognized format function type'

compare: http://lists.osgeo.org/pipermail/grass-user/2010-January/054025.html

best regards Helmut

Change History (5)

comment:1 in reply to:  description ; Changed 10 years ago by glynn

Priority: normalminor

Replying to hellik:

unfortunately I wasn't all the building time in front of my laptop and couldn't catch all the messages,

That's why you should redirect output to a file, e.g.

make &> build.log

but I've seen in a few-modules (raster and vector etc.; i.e. nviz, v.what, v.out.pov, v.support, v.transform, v.to.db ) following warnings:

`libintl_printf' is an unrecognized format function type'

I count 77 distinct occurrences (each one will generate multiple warnings, one for each occurrence of __attribute__((format(printf,...))) in gisdefs.h or G3d.h.

Downgrading to "minor", as these are only warnings.

They could be silenced by re-defining attribute to an empty macro (as we do if GNUC isn't defined).

We could eliminate the redirection to libintl_printf, as none of the translations are currently using the %1$ syntax (which the MSVCRT printf() doesn't support, hence the redirection). However, chances are we'll forget to re-instate it if we ever need it.

comment:2 Changed 9 years ago by hellik

Keywords: wingrass added

comment:3 Changed 9 years ago by hellik

Component: PackagingCompiling

comment:4 in reply to:  1 Changed 9 years ago by hellik

Replying to glynn:

We could eliminate the redirection to libintl_printf, as none of the translations are currently using the %1$ syntax (which the MSVCRT printf() doesn't support, hence the redirection). However, chances are we'll forget to re-instate it if we ever need it.

so maybe closing as wontfix?

Helmut

comment:5 Changed 8 years ago by hellik

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.