build watch on windows

dealerpriest

Hello. I'm trying to use the build_project.sh script with the -b and --watch flags. Unfortunately the script doesn't seem to react to any changes in the source files. I think it might be related to me being on windows. I've tried with both msys and git-bash, without success.
Is it in fact so that the watch functionality won't work with these bash emulators? Would be really nice to have this functionality!

bash build_project.sh ../../bela_projects/octave -b --watch

Gives the following output in msys:

Checking the board is up and running at root@192.168.7.2...done
Stop running process...
Project `octave' already exists on the board, do you want to overwrite it? (y/n)
: y
Copying new source files to BeagleBone...using rsync...
Building and running project...
Running /root/Bela/projects/octave/octave  in a screen
Waiting for changes in ../../bela_projects/octave, or press ctrl-c to terminate

giuliomoro

Hmm it works for my on Windows 10 with git-bash (mingw32). However I realized a bug that I just fixed on master, which I think is unrelated to your issue, but please update your repo before continuing.

If the problem persists, try to add the following line close to the top (i.e..: after the first line) of build_project.sh:

set -x

then run the script again. This time the log should be much more verbose. Copy and paste it here.

dealerpriest

Tried with logging enabled using the latest version from master branch. Log output attached.
Still no reactions when saving to the files. I am on windows 8.

$ bash build_project.sh ../../bela_projects/octave -b --watch
++ dirname build_project.sh
+ SCRIPTDIR=.
+ '[' -z . ']'
+ SCRIPTDIR=./
+ . ./.bela_common
++ '[' -f /c/Users/tii/.belarc ']'
++ '[' -z '' ']'
++ BBB_USER=root
++ '[' -z '' ']'
++ BBB_HOSTNAME=192.168.7.2
++ '[' -z '' ']'
++ BBB_ADDRESS=root@192.168.7.2
++ '[' -z '' ']'
++ BBB_BELA_HOME=/root/Bela
++ '[' -z '' ']'
++ RUN_PROJECT=1
++ '[' -z '' ']'
++ COMMAND_ARGS=
++ '[' -z '' ']'
++ BBB_PROJECT_HOME=/root/Bela/projects/
++ '[' -z '' ']'
++ RUN_MODE=foreground
++ '[' -z '' ']'
++ USE_RSYNC=1
++ sleep 0.0001
++ SLEEP_SUPPORTS_FLOAT=1
++ '[' -z '' ']'
++ '[' 1 -eq 1 ']'
++ DEFAULT_SLEEP_WHEN_WATCHING=0.5
+++ basename build_project.sh
++ THIS_SCRIPT=build_project.sh
++ case_scripts_options='
run_options{
                -c)
                        shift;
                        COMMAND_ARGS="$1";
                ;;
                -b)
                        RUN_MODE=screen;
                ;;
                -f)
                        RUN_MODE=foreground;
                ;;
                -s)
                        RUN_MODE=screenfg;
                ;;
                }
build_options{
                -n)
                        RUN_PROJECT=0;
                ;;
                -p)
                        shift;
                        BBB_PROJECT_NAME="$1";
                ;;
                --clean)
                        BBB_MAKEFILE_OPTIONS="$BBB_MAKEFILE_OPTIONS projectclean";
                ;;
                --force)
                        FORCE=1
                ;;
                -m)
                        shift;
                        BBB_MAKEFILE_OPTIONS="$BBB_MAKEFILE_OPTIONS $1";
                ;;
                --watch)
                        WATCH=1
                ;;
}
end_options{
                --help|-h|-\?)
                        usage;
                        exit 0;
                ;;
                -*)
                        echo Error: unknown option $1
                        usage_brief
                        exit 1;
                ;;
}
'
+ RUN_MODE=foreground
+ WATCH=0
+ HOST_SOURCE_PATH=
+ FORCE=0
+ EXPERT=0
+ '[' -n ../../bela_projects/octave ']'
+ case $1 in
+ '[' -z '' ']'
+ HOST_SOURCE_PATH=../../bela_projects/octave
+ shift
+ '[' -n -b ']'
+ case $1 in
+ RUN_MODE=screen
+ shift
+ '[' -n --watch ']'
+ case $1 in
+ WATCH=1
+ shift
+ '[' -n '' ']'
+ '[' 0 -eq 1 ']'
+ '[' -z ../../bela_projects/octave ']'
+ FIND_STRING='find ../../bela_projects/octave -maxdepth 1 -type f '
+ EXTENSIONS_TO_FIND='\.cpp\|\.c\|\.S\|\.pd\|\.scd'
++ find ../../bela_projects/octave -maxdepth 1 -type f
++ grep '\.cpp\|\.c\|\.S\|\.pd\|\.scd'
+ FOUND_FILES='../../bela_projects/octave/render.cpp
../../bela_projects/octave/utility.cpp'
+ '[' -z '../../bela_projects/octave/render.cpp
../../bela_projects/octave/utility.cpp' ']'
+ '[' -z ']'
+++ cd ../../bela_projects/octave
+++ pwd
++ basename /c/bela_projects/octave
+ BBB_PROJECT_NAME=octave
+ BBB_PROJECT_FOLDER=/root/Bela/projects//octave
+ BBB_NETWORK_TARGET_FOLDER=root@192.168.7.2:/root/Bela/projects//octave
+ '[' 0 -eq 0 ']'
+ check_board_alive
+ printf 'Checking the board is up and running at root@192.168.7.2...'
Checking the board is up and running at root@192.168.7.2...+ ssh -o StrictHostKeyChecking=no -o ConnectTimeout=5 root@192.168.7.2 exit
+ printf 'done\n'
done
+ '[' 0 -eq 0 ']'
+ set_date
++ date '+%Y%m%d %T %z'
+ ssh root@192.168.7.2 'date -s "20170820 19:09:07 +0200" > /dev/null; '
+ echo 'Stop running process...'
Stop running process...
+ ssh root@192.168.7.2 make QUIET=true --no-print-directory -C /root/Bela stop
+ '[' 0 -eq 1 ']'
+ check_project_exists_prompt octave
+ check_project_exists octave
+ '[' -z octave ']'
+ ssh root@192.168.7.2 '[ -d /root/Bela/projects//octave ]'
+ return 0
+ printf 'Project `octave'\'' already exists on the board, do you want to overwrite it? '
Project `octave' already exists on the board, do you want to overwrite it? + interactive 1
+ '[' -z 1 ']'
+ COUNT=1
+ STR=
+ '[' 1 -eq 1 ']'
+ STR='(y/n): '
+ printf '(y/n): '
(y/n): + read REPLY
y
+ '[' 1 -eq 1 ']'
+ case $REPLY in
+ REPLY=1
+ break
+ _RET1=1
+ return 0
+ '[' 1 -eq 0 ']'
+ TMP_DIR=.//../tmp
+ reference_time_file=.//../tmp/.timeoctave
+ check_rsync
+ '[' 1 -eq 0 ']'
++ which rsync
+ '[' -z '' ']'
+ return 1
+ RSYNC_AVAILABLE=0
+ uploadBuildRun
+ '[' 1 -eq 1 ']'
+ mkdir -p .//../tmp
+ touch .//../tmp/.timeoctave
+ printf 'Copying new source files to BeagleBone...'
Copying new source files to BeagleBone...+ '[' 0 -eq 0 ']'
+ echo 'using scp...'
using scp...
+ echo 'Cleaning the destination folder...'
Cleaning the destination folder...
+ ssh root@192.168.7.2 'rm -rf "/root/Bela/projects//octave"; mkdir -p "/root/Bela/projects//octave"'
+ echo 'Copying the project files'
Copying the project files
+ scp -r ../../bela_projects/octave/render.cpp ../../bela_projects/octave/utility.cpp root@192.168.7.2:/root/Bela/projects//octave
render.cpp                                                100% 3292     3.2KB/s   00:00
utility.cpp                                               100%    0     0.0KB/s   00:00
+ '[' 0 -ne 0 ']'
+ MAKE_COMMAND='make --no-print-directory QUIET=true -C /root/Bela PROJECT='\''octave'\'' CL='\'''\'' '
+ '[' 1 -eq 0 ']'
+ echo 'Building and running project...'
Building and running project...
+ case_run_mode
+ case $RUN_MODE in
+ ssh root@192.168.7.2 'make --no-print-directory QUIET=true -C /root/Bela PROJECT='\''octave'\'' CL='\'''\''  runscreen'
Building render.cpp...
 ...done

Building utility.cpp...
 ...done

Linking...
 ...done
Running /root/Bela/projects/octave/octave  in a screen
+ '[' 1 -ne 0 ']'
+ true
+ echo 'Waiting for changes in ../../bela_projects/octave, or press ctrl-c to terminate'
Waiting for changes in ../../bela_projects/octave, or press ctrl-c to terminate
+ CORE_DIR=.//../core
+ INCLUDE_DIR=.//../include
+ wait_for_change ../../bela_projects/octave .//../tmp/.timeoctave
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ SLEEP=0.5
+ folder_has_changed ../../bela_projects/octave .//../tmp/.timeoctave ''
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ FILTER=.
+ find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave
+ grep .
+ return 1
+ sleep 0.5
+ folder_has_changed ../../bela_projects/octave .//../tmp/.timeoctave ''
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ FILTER=.
+ find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave
+ grep .
+ return 1
+ sleep 0.5
+ folder_has_changed ../../bela_projects/octave .//../tmp/.timeoctave ''
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ FILTER=.
+ find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave
+ grep .
+ return 1
+ sleep 0.5
+ folder_has_changed ../../bela_projects/octave .//../tmp/.timeoctave ''
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ FILTER=.
+ find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave
+ grep .
+ return 1
+ sleep 0.5
+ folder_has_changed ../../bela_projects/octave .//../tmp/.timeoctave ''
+ '[' -z .//../tmp/.timeoctave ']'
+ '[' -z '' ']'
+ FILTER=.
+ find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave
+ grep .
+ return 1
+ sleep 0.5

giuliomoro

Right, the line

find ../../bela_projects/octave -type f -not -path '*/\.*' -newer .//../tmp/.timeoctave

is searching for files in ../../bela_projects/octave -type that are newer than.//../tmp/.timeoctave. Maybe there is something wrong with your find program?

Could you open another terminal and while the above script is running, run (assuming you are in the same working directory):

ls -lh ./../tmp/.timeoctave ../../bela_projects/octave

run it again after editing one or more files in the ../../bela_projects/octave folder.

Also, maybe send a dump of find --help (or man find if man is available on your system)?

giuliomoro

I think that this was a bug that was not Windows-specific. I stumbled upon it the other night and I fixed it.
It was triggered depending on how you were passing the path to the project folder.
Can you try again with the new master branch (note: you need to update the scripts you are using on the host).

dealerpriest

Sorry for late response!
Unfortunately I no longer have the same computer available to see if the problem is fixed. But I can confirm that the -watch option works fine on my current setup running Windows 10.
Thanks for all the efforts!