search

muschellij2 / read.cwa

Convert 'Axtivity' 'CWA' Files
GNU General Public License v3.0
0 stars 1 forks source link

Use a local definition of timegm() for better portability. #2

Open danielgjackson opened 3 months ago

danielgjackson commented 3 months ago

Uses a local definition of timegm() for better portability, fixing issue #1.

muschellij2 commented 3 months ago

I've made changes to master. Can you pull and re-merge? I think there are warnings wrt sprintf

❯ checking compiled code ... WARNING File ‘read.cwa/libs/read.cwa.so’: Found ‘___sprintf_chk’, possibly from ‘sprintf’ (C) Object: ‘main.o’

Compiled code should not call entry points which might terminate R nor write to stdout/stderr instead of to the console, nor use Fortran I/O nor system RNGs nor [v]sprintf.

See ‘Writing portable packages’ in the ‘Writing R Extensions’ manual.