Great work 👍 Wondering how we can prioritise this for the next release 🤔

Great work 👍 Wondering how we can prioritise this for the next release 🤔

Thanks! You might want to post your interest on the parent issue, it might get more visibility there.

For my changes:

• Called apiDump
• Changed libs.kotlinx.io from implementation to api as you cannot add the library without them ( was going to do the same for libs.kotlinx.datetime but I want to prevent version mismatches as some people already use datetime)
• Re-ordered variables in RollingFileLogWriter so that the non-optional is first
• Added new LogWriter to sample app, and adding mobileMain sourceset to handle new LogWriter
• renaming logPath to logFilePath and logPrefix to logFileName

Very nice work, however we're running into cinterop issues when adding this change to the sample app, which causes a build error when testing. We want to ensure it works and builds properly before merging it in, so I'm holding off until we can fix this issue. The issues arise when building the sample app. At first it was an issue with the cinterop:

e: KLIB resolver: Could not find "co.touchlab:kermit-core-cinterop-os_log" in [/Users/kevinschildhorn/Documents/GitHub/Kermit/samples/sample, /Users/kevinschildhorn/.konan/klib, /Users/kevinschildhorn/.konan/kotlin-native-prebuilt-macos-aarch64-2.0.21/klib/common, /Users/kevinschildhorn/.konan/kotlin-native-prebuilt-macos-aarch64-2.0.21/klib/platform/ios_arm64]

However recently the error is:

> Following dependencies exported in the podDebugFramework binary are not specified as API-dependencies of a corresponding source set:
  
  Files: [/Users/kevinschildhorn/.gradle/caches/modules-2/files-2.1/co.touchlab/kermit-simple-iosarm64/2.0.4/704ee2e118d3850b5d4eb1b3148a2f306d0e9917/kermit-simple.klib]
  Files: [/Users/kevinschildhorn/.gradle/caches/modules-2/files-2.1/co.touchlab/kermit-simple-iosarm64/2.0.4/704ee2e118d3850b5d4eb1b3148a2f306d0e9917/kermit-simple.klib]
  
  Please add them in the API-dependencies and rerun the build.

Very nice work, however we're running into cinterop issues when adding this change to the sample app, which causes a build error when testing. We want to ensure it works and builds properly before merging it in, so I'm holding off until we can fix this issue.

How can I reproduce this? I've done build on the sample project without any errors locally.

Nevermind the issue was resolved, an issue with default heriarchy.

Are we handling the exceptions from this writer correctly? We've noticed there might be restrictions on ios writing to file when the device is locked, which causes this exception. I've opened #468 for this

kotlinx.io.IOException: fflush failed with errno 1 (Operation not permitted)
    at 0   debug.dylib                  0x11742031b        kfun:kotlinx.io.files.throwIOExceptionForErrno#internal + 547 
    at 1   debug.dylib                  0x11741fff3        kfun:kotlinx.io.files.FileSink#flush(){} + 239 
    at 2  debug.dylib                  0x11741415b        kfun:kotlinx.io.RealSink#flush(){} + 435 
    at 3   debug.dylib                  0x11767fc8f        kfun:co.touchlab.kermit.io.$writerCOROUTINE$0.invokeSuspend#internal + 1515