search

tarm / serial

BSD 3-Clause "New" or "Revised" License
1.61k stars 452 forks source link

Update README.md #19

Closed jeffallen closed 9 years ago

jeffallen commented 9 years ago

Fixed package name confusion.

jeffallen commented 9 years ago

Nevermind. But that's just whack that the repo name and the package name are not the same. Way confusing.

tarm commented 9 years ago

I agree with you, but it would break people's code to change the package name or import path now.

This repo (and naming) predate the go tool as you can tell from the README.

I am considering making a new package github.com/tarm/serial that would fix the import path issue and also allow a few backwards-incompatible changes (since the import path would then be different it wouldn't break people's code).

tarm commented 9 years ago

OK, I just renamed the package, made a backwards incompatible change, and then restore goserial for backwards compatibility.

The new package path is github.com/tarm/serial

jeffallen commented 9 years ago

Thx. Two thumbs up!

On Thu, Feb 26, 2015 at 3:56 AM, tarm notifications@github.com wrote:

OK, I just renamed the package, made a backwards incompatible change, and then restore goserial for backwards compatibility.

The new package path is github.com/tarm/serial

— Reply to this email directly or view it on GitHub https://github.com/tarm/serial/pull/19#issuecomment-76113729.