#aerofoil

/

      • crEpE
        But thats just my opinion
      • ke7ofi
        I asked for your opinion.
      • crEpE
        Does that answer your question?
      • ke7ofi
        It does. Lets include that in preflight.
      • standard: vanilla to d, desktop to D
      • newbie: desktop to d, vanilla to D
      • crEpE
        Alright sounds good
      • ke7ofi
        Do you have time to at least email a copy?
      • crEpE
        It's pretty bare right now but sure. Still has some work to do
      • ke7ofi
        When you get the chance, please go to the Aerofoil Members page (on GitHub) and set your visibility to public.
      • crEpE
        Done
      • ke7ofi
        thanks
      • crEpE
        Question.. in autobot, line 54, is that supposed to echo $CODENAME into the config or the string which $CODENAME represents
      • ke7ofi
        Let me take a look.
      • Go ahead.
      • Do you see a problem?
      • Its supposed to echo to the config.
      • crEpE
        I've cleaned up a few more things in there as well, so if you want I can change that and submit a PR for it
      • No a major problem. Double brackets are preferred for the equates and I think you wanted to use double quotes " instead of single ' on the other lines
      • ke7ofi
        What do you mean by equates?
      • crEpE
        line 36 for example
      • when you are checking that files exist
      • ke7ofi
        Ill change that.
      • The single quotes were deliberate.
      • crEpE
        Alrighty, its about lunch time for me
      • Be back later
      • ke7ofi
        bye
      • crEpE
        Graves?
      • ke7ofi
        `
      • instead of '
      • What do they do?
      • crEpE
        $(cat blah)
      • instead of `cat blah`
      • ke7ofi
        and thats just for POSIX compliance, or is there any other difference?
      • I learned everything I know about shell-scripting from the command-line and Nathan, so Im a little under-informed. Do you know of any good resources besides the bash manual?
      • crEpE
        Pretty much just POSIX compliance / looks a little cleaner
      • I manually use the manual and just stack overflow haha
      • mainly*
      • ke7ofi
        That typo was a beautiful pun.
      • crEpE
        haha
      • Shall I submit another pull request for that?
      • My bad =(
      • Uh oh. My pull request has two errors. line 30 and line 36. There should be a space before the last ]]
      • ke7ofi
        sure
      • In fact, feel free to push straight to master.
      • Its slightly lax, but fine if youre sure.
      • crEpE
        Crap submitted the PR before reading that last part
      • ke7ofi
        Dont worry about it.
      • crEpE
        Sorry for being a git noob, but whats up with those <<<<< HEAD statements in aerobot? Did I screw something up?
      • ke7ofi
        Where are they? Are they actually in a file, or just the repos metadata?
      • Dont worry about ignorance. Youve seen my incompetence in Bash. Were both learning.
      • You did screw something up. Ill fix it.
      • crEpE
        What did I do?
      • ke7ofi
        Im not sure. You probably middle-clicked or something.
      • crEpE
        Interesting. I'll be sure to make the request properly next time
      • ke7ofi
        Actually, it may have been automatically inserted when I resolved the conflicts.
      • In either case, itll be fixed in a couple of minutes.
      • fixed now
      • That wasnt your fault. It was partially mine and mostly Gits.
      • unless I *really* dont know how to use Git
      • crEpE
        What went wrong?
      • ke7ofi
        Git inserted summaries (uncommented, for some reason) into the code instead of the commits or metadata. I have no idea why this behaviour exists.
      • Ill probably be the first person to send Torvalds an angry email instead of the other way around.
      • crEpE
        haha