Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

When attaching, set numeric username to NOUID if not using dotu. #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

When attaching, set numeric username to NOUID if not using dotu. #9

wants to merge 2 commits into from

Commits on Aug 17, 2015

  1. When attaching, set numeric username to NOUID it not using dotu. 

    Missing else clause.  See:
https://github.com/9fans/plan9port/blob/master/src/lib9/convM2S.c#L106-L122

Fixes issue rminnich#8.
    @mbucc
    mbucc committed Aug 17, 2015
    Configuration menu
    Copy the full SHA
    d8aa840 View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2015

  1. Unbreak tests; set Dotu=true server so version response is 9P2000.u 

    Otherwise,

  1. the server version is 9P2000,
  2. clnt.Dotu is set to false,
  3. Srv.attach tries to get user from Uname2User(),
  4. which returns nil in OsUsers and
  5. so attach fails.
    @mbucc
    mbucc committed Aug 22, 2015
    Configuration menu
    Copy the full SHA
    12fc863 View commit details
    Browse the repository at this point in the history