-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Scripting Session #2
Comments
Yep so far ive covered variables and for loops (i still need to finish). From what ive seen in the previous materials that John sent me they cover permissions, but I can easily cover that and basename stuff as well |
For sure. I've just finished Section2 & was trying to figure which bits I include/exclude. Feedback welcome, but I think it's looking pretty good. |
Morning Boys, sounds as if you two are all over it but if not, let me know how I can help! |
Is it possible for someone to introduce |
Ive included them in cheatsheet, but that's still work in progress. I can introduce them as incidentals to example questions or exercises. |
They're introduced in Session 2: https://big-sa.github.io/BASH-Intro-2018/notes/2_filters.html#this-session |
Ok ive done a rough draft of the entire bash scripting session. Might be a bit raw, so if someone has time, are you able to look through it and suggest any additions/subtractions or interesting exercises? Still quite a bit of time left to clean this up nicely |
Well done Jimmy! I'll take a look sometime over the weekend and will get back to you.
Cheers J
From: Jimmy Breen [mailto:[email protected]]
Sent: Thursday, 12 April 2018 21:06
To: BIG-SA/BASH-Intro-2018 <[email protected]>
Cc: John Toubia <[email protected]>; Assign <[email protected]>
Subject: Re: [BIG-SA/BASH-Intro-2018] Scripting Session (#2)
Ok ive done a rough draft of the entire bash scripting session. Might be a bit raw, so if someone has time, are you able to look through it and suggest any additions/subtractions or interesting exercises? Still quite a bit of time left to clean this up nicely
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub<#2 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFRyGYv_rj0e9LFS2ZY4W3ab32Xpld25ks5tnzwmgaJpZM4TN4Qe>.
|
Hey Again,
Sorry, I didn't get to finish this over the weekend! I just looked at the last bits and pushed all my changes. here are some additional notes for you to address as well;
line 43 - bringing down the particle pollution data. the user should be in a newly created folder called "BashWk4" which is empty. "cd ./BASH-Intro-2018/files/" gives an error? This might be intentional and if they learnt anything from the previous weeks they should be able to create these directories.
line 75-76 - in the basic_example_1.sh script. Added a note just in case they used a different download location and modified the cd command to be absolute from home (cd ~/BashWk4/BASH-Intro-2018/files)
line 90 - questions about first script. They might not realise that the output gets saved in 'ADL07p' so could ask that too?
line 148 - user and group settings. Added a little more detail to that sentence for clarification
line 153 - after changing permisions. you state "As a security measure, Linux doesn’t allow you to execute a script from within the same directory so to execute it enter the following: ./scriptname.sh". Not really an issue as it doesn't matter a great deal but you can actually call a script from within the directory as long as that directory is in your $PATH. Otherwise you specifically need to provide relative or absolute path to script which is why './' works.
line 175 - second bash script iteration. 'cd' command is commented out possibly because we expect them to be in the right directory. Maybe should be an absolute path though?
line 196 - variables. slight tweak about the variables 'ME' and 'MESSAGE' they had not been used yet.
line 206 - minor mods
line 237 - welldone.sh. not sure about Mac and Windows but on linux that line should be something like this; echo -e "Hello ${ME}\n\"${MESSAGE}\"\nWell Done!"
line 272 - count_lines.sh. I don't know jack about this data but I think each line in the file is a reading per hour not month? If so the print line "The number of months contained in the data is ..." should be changed.
line 284 - exercise. need to finish that sentence
line 343 - minor mods
line 411 - basic_example_3.sh. fixed variable name (WORKING_DIRECTORY --> $WORKING_DIR)
Everything else worked out of the box and the flow is brilliant! Well done Jimmy!!
Cheers
John
From: Jimmy Breen [mailto:[email protected]]
Sent: Thursday, 12 April 2018 21:06
To: BIG-SA/BASH-Intro-2018 <[email protected]>
Cc: John Toubia <[email protected]>; Assign <[email protected]>
Subject: Re: [BIG-SA/BASH-Intro-2018] Scripting Session (#2)
Ok ive done a rough draft of the entire bash scripting session. Might be a bit raw, so if someone has time, are you able to look through it and suggest any additions/subtractions or interesting exercises? Still quite a bit of time left to clean this up nicely
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub<#2 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFRyGYv_rj0e9LFS2ZY4W3ab32Xpld25ks5tnzwmgaJpZM4TN4Qe>.
|
Thanks John! Tweaked a few things again, but will look through again later this week |
Too easy Champ! I’ll give it another run through later in the week as well.
From: Jimmy Breen <[email protected]>
Sent: Monday, 16 April 2018 18:36
To: BIG-SA/BASH-Intro-2018 <[email protected]>
Cc: John Toubia <[email protected]>; Assign <[email protected]>
Subject: Re: [BIG-SA/BASH-Intro-2018] Scripting Session (#2)
Thanks John! Tweaked a few things again, but will look through again later this week
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub<#2 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFRyGd6PYlIY0wQ7r5XXsiveswJsMd8vks5tpF73gaJpZM4TN4Qe>.
|
Hey Jimmy,
Just thinking about that Scripting Session. I'm thinking a nice clear description of things like the following might be good
basename
anddirname
for
loopsI don't think we'll have covered these by that week, so it might make a good start. Unless Paul will cover for loops in the sed/awk sessions?
The text was updated successfully, but these errors were encountered: