Edit report at https://bugs.php.net/bug.php?id=14615&edit=1

 ID:                 14615
 Updated by:         yohg...@php.net
 Reported by:        timvandermolen at hotmail dot com
 Summary:            PHP log does not contain proper CRLF's
-Status:             Open
+Status:             Closed
 Type:               Feature/Change Request
 Package:            PHP options/info functions
 Operating System:   Windows
 PHP Version:        *
-Assigned To:        
+Assigned To:        yohgaki
 Block user comment: N
 Private report:     N

 New Comment:

At least 5.5 uses CR/LF under windows.


Previous Comments:
------------------------------------------------------------------------
[2001-12-20 04:52:46] hholz...@php.net

now this is strange ...

the following snippet from php4/main/main.c
(around line 300) does the actual logging
and as it does *not* open the logfile in
binary mode it should automaticly convert
'\n' to whatever the local platform prefers???

  log_file = VCWD_FOPEN(PG(error_log), "a");
  if (log_file != NULL) {
    time(&error_time);
    strftime(error_time_str, 128
            , "%d-%b-%Y %H:%M:%S"
            , php_localtime_r(&error_time, &tmbuf)); 
    fprintf(log_file, "[%s] ", error_time_str);
    fprintf(log_file, "%s", log_message);
    fprintf(log_file, "\n");
    fclose(log_file);
    return;
  }


------------------------------------------------------------------------
[2001-12-20 04:36:06] mfisc...@php.net

Reopen, leave Type: to Feature Request.

------------------------------------------------------------------------
[2001-12-20 04:28:40] timvandermolen at hotmail dot com

"but i'm not really sure what log we are talking about here"

I am talking about the PHP log file which can be defined with the directive 
"error_log = ..." in the PHP.INI file.

------------------------------------------------------------------------
[2001-12-20 04:23:54] timvandermolen at hotmail dot com

There's nothing wrong with the editor. I have used Notepad to view the PHP log.

I have also viewed the PHP log with Visual C++ which *converts* (note the word 
convert) the LF to CRLF. Appearently, CRLF is the preferred format in Windows 
for new lines above LF.

Besides, CRLF is the new line format which practically every editor in Windows 
supports.

And besides everything: CRLF *is* the Windows newline, LF is not. So it would 
not make sense to use LF as a new line in Windows environments.

------------------------------------------------------------------------
[2001-12-20 04:23:14] hholz...@php.net

well, different systems *have* different default
line ending styles, so we might want to write
whatever is considered 'native' on a certain
platform

but i'm not really sure what log we are talking
about here, as the default log mechanism (as i
know it on Linux/Apache) is to pass log messages
to the webserver for logging? 

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=14615


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=14615&edit=1

Reply via email to