Opened 5 years ago

Closed 4 years ago

#2745 closed defect (fixed)

i.cluster: report file has wrong linebreaks on Windows

Reported by: neteler Owned by: martinl
Priority: normal Milestone: 7.0.3
Component: Imagery Version: svn-releasebranch70
Keywords: i.cluster Cc: grass-dev@…
CPU: Unspecified Platform: MSWindows 7

Description

The reportfile in i.cluster (see manual page for example) is stored with UNIX style newlines. At time unix2dos is needed.

What is the needed change to change i.cluster to write "new lines" chars in a way that the resulting ASCII file is properly formatted also on Windows?

We have done this elsewhere (v.out.ascii?), but I cannot find it.

Change History (6)

comment:1 Changed 5 years ago by neteler

Milestone: 7.0.27.0.3

Ticket retargeted after milestone closed

comment:2 Changed 4 years ago by martinl

Cc: grass-dev@… added
Owner: changed from grass-dev@… to martinl
Status: newassigned

comment:3 in reply to:  description ; Changed 4 years ago by martinl

Replying to neteler:

The reportfile in i.cluster (see manual page for example) is stored with UNIX style newlines. At time unix2dos is needed.

What is the needed change to change i.cluster to write "new lines" chars in a way that the resulting ASCII file is properly formatted also on Windows?

Fixed in trunk (r67276:67277)

We have done this elsewhere (v.out.ascii?), but I cannot find it.

No, now part of gislib (r67275). Also G7:v.out.ascii and related subroutines from Vlib fixed (r67278).

comment:4 Changed 4 years ago by martinl

You can test latest winGRASS daily builds (build no. 19 on x86_64 or 180 on x86). Then I will backport changes.

comment:5 in reply to:  3 Changed 4 years ago by martinl

Replying to martinl:

Fixed in trunk (r67276:67277) No, now part of gislib (r67275). Also G7:v.out.ascii and related subroutines from Vlib fixed (r67278).

I took liberty to backport these changes as bugfixes (r67364:67366). For testing please use daily builds of 7.0.3svn standalone installer (1,2)

(1) https://wingrass.fsv.cvut.cz/x86_64/grass70/

(2) https://wingrass.fsv.cvut.cz/x86/grass70/

comment:6 Changed 4 years ago by martinl

Resolution: fixed
Status: assignedclosed

I have tested fresh build of 7.0.3svn both G7:i.cluster and G7:v.out.ascii. Seems to be OK. Closing the ticket, feel free to re-open if needed.

Note: See TracTickets for help on using tickets.