<div dir="ltr"><div>Hi, Kostiantyn</div><div><br></div><div>Thank you for the quick reply.</div><div><br></div><div>You guess correctly.</div><div>I'm really sorry that I couldn't explain it well.</div><div><br></div><div>The reason why I asked that I didn't know about C99 in spite of looking up it.</div><div>And, the reason why I reported <a href="https://bugs.swift.org/browse/SR-1224">https://bugs.swift.org/browse/SR-1224</a> is what Kostiantyn said.</div><div><br></div><div>> we should provide better error handling.</div><div><br></div><div>Best regards,</div><div>JPMartha</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-04-15 5:23 GMT+09:00 Kostiantyn Koval <span dir="ltr"><<a href="mailto:konstantin.koval1@gmail.com" target="_blank">konstantin.koval1@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Hi Daniel and JPMartha</div><div><br></div>From what I understand we we should provide better error handling. We should check is the package name is correct when run swift build --init<div>If it’s not then descriptive show error message and don’t create a Package.</div><div><br><div>Did I understood it correctly JPMartha?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>- Kostiantyn</div></font></span><div><div class="h5"><div><div><br><div><blockquote type="cite"><div>On 14 Apr 2016, at 17:54, Daniel Dunbar via swift-build-dev <<a href="mailto:swift-build-dev@swift.org" target="_blank">swift-build-dev@swift.org</a>> wrote:</div><br><div><div style="word-wrap:break-word">Hi JPMartha,<div><br></div><div>I am not 100% sure what your question is. Is it whether this name should be valid, or just whether we should diagnose it earlier?</div><div><br></div><div>It is an invalid "C99 Extended Identifier", which is the technical specification these names need to follow (a digit is not a valid initial character).</div><div><br></div><div> - Daniel</div><div><br><div><blockquote type="cite"><div>On Apr 13, 2016, at 10:56 PM, Martha JP via swift-build-dev <<a href="mailto:swift-build-dev@swift.org" target="_blank">swift-build-dev@swift.org</a>> wrote:</div><br><div><div dir="ltr"><div>Hi, I'm JPMartha.</div><div><br></div><div>Can I ask about module name?</div><div><br></div><div>The following error has occured when package's name or module's name consists of figures only. Is it invalid as the error message says?</div><div><br></div><div>Error:</div><div>```</div><div>fatal error: Invalid module name: file /Users/buildnode/jenkins/workspace/oss-swift-package-osx/swiftpm/Sources/PackageType/c99name().swift, line 202</div><div>```</div><div><br></div><div>I saw `Sources/PackageType/c99name().swift`, then I found the following document.</div><div><br></div><div>/**</div><div> Removes characters from name that are invalid in C99 module-names.</div><div>*/</div><div><br></div><div>So, I checked C99. Also, I saw the API Design Guideline and the swift-build-dev Archives.</div><div>But I'm not sure that it is invalid.</div><div><br></div><div>Even though I miss a naming rule, I think this error should been prevented beforehand.<br></div><div>So, I reported <a href="https://bugs.swift.org/browse/SR-1224" target="_blank">https://bugs.swift.org/browse/SR-1224</a>.</div><div><br></div><div>Best regards,</div><div>JPMartha</div></div>
_______________________________________________<br>swift-build-dev mailing list<br><a href="mailto:swift-build-dev@swift.org" target="_blank">swift-build-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-build-dev" target="_blank">https://lists.swift.org/mailman/listinfo/swift-build-dev</a><br></div></blockquote></div><br></div></div>_______________________________________________<br>swift-build-dev mailing list<br><a href="mailto:swift-build-dev@swift.org" target="_blank">swift-build-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-build-dev" target="_blank">https://lists.swift.org/mailman/listinfo/swift-build-dev</a><br></div></blockquote></div><br></div></div></div></div></div></div></blockquote></div><br></div>