-
Notifications
You must be signed in to change notification settings - Fork 98
Conversation
androdeb
Outdated
########################################################## | ||
# GIT-PULL | ||
########################################################## | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lose these extra new lines, and comments. Lets keep it as small a change as possible. The code is self-documenting.
androdeb
Outdated
exit 0 | ||
fi | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please no extra new lines
androdeb
Outdated
@@ -95,6 +127,7 @@ if [ ! -z ${ASHELL+x} ]; then | |||
exit 0 | |||
fi | |||
|
|||
|
|||
########################################################## |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why newline?
androdeb
Outdated
shift || true | ||
fi | ||
|
||
if [ "2" -le "$#" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not: if [ $# -le 2 ]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to check if 2 or more arguments are remaining (multiple files / local destination specified), I guess it could be:
if [$# -gt 1 ]
androdeb
Outdated
done | ||
else | ||
FILES["0"]=/data/androdeb/debian/$1 | ||
shift || true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this shift needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to consume the argument because the last argument (local destination) is passed through below
Joel, I refactored the function to make it easier to understand and got rid of the extra lines. Mind taking a look? |
There was already "androdeb pull" command, which did a git pull. This was renamed to "androdeb git-pull". The new command makes it easier to retrieve files from the androdeb filesystem. The same options as "adb pull" are supported. Signed-off-by: Erick Reyes <erickreyes@google.com>
There was already "androdeb pull" command, which did a git pull. This was renamed to "androdeb git-pull". The new command makes it easier to retrieve files from the androdeb filesystem. The same options as "adb pull" are supported.