Remove second call to getline

[jonboiser]

Testing learningequality/kolibri#8083, it seems like calling getline twice ends up discarding the first two lines, so that when we read the line, we're actually getting the third line. So this just removes the second call.

[radinamatic]

We seem to be closer with the build from this PR, but not completely there yet:

• After the installation, Kolibri.exe (taskbar launcher) still announces that the server will be running on the port 4xxxx, but the browser opens the correct port 8080.
• However, if the browser is closed, and reopened with the Load in a browser option from the launcher menu, browser loads the page at the port 4xxxx 😕
• Similarly, if the server is stopped and restarted from the desktop shortcut, or the system is restarted, browser will still load the page on the port 4xxxx.

[radinamatic]

Tested on Windows 7, but it will almost certainly be the same on Win 10.

[micahscopes]

That 49189 still appears to be line #3. I wonder if there's some kind of build cache issue that's really the cause of our woes here?

[rtibbles]

Running a minimal version of the CPP logic in Windows against a PID file suggests to me that our work here has not made it downstream. @micahscopes's edit should have correctly returned the second line, and @jonboiser's edit would be returning the first line.

[rtibbles]

That said - looking at the build job on buildkite, it does seem to be fetching the proper commit :/

[rtibbles]

The only thing I can think of here is that somehow the updating of the PID file by Kolibri is causing inconsistent reads by the CPP application. Going to do some more testing locally to see if repeated reads change while Kolibri is booting up.