On 16/06/18 15:10, RobH wrote:
> On 16/06/18 14:58, RobH wrote:
>> On 16/06/18 13:59, Dennis Lee Bieber wrote:
>>> On Sat, 16 Jun 2018 12:49:30 +0100, RobH 
>>> declaimed the
>>> following:
>>>
>>>
>>>> from os.path import join < I added this
>>>>
>>>     Why? Was typing    os.path.join    too much effort?
>>
>> No it wasn't actually, as I said in another reply on here, I have been
>> reading up on the shutil module, and I thought to try some of the
>> examples I looked at.
>>
>>>
>>>> src = ("RAMname")< and this
>>>> dst = ("NASname")< this
>>>
>>>     These lines do nothing useful since you never use the names src
>>> and dst
>>> later (except to replace the contents and again not use them)
>>
>> No I have removed them now
>>>
>>>
>>>> while True:
>>>>      # Wait for motion being detected
>>>>      pir.wait_for_motion()
>>>>      triggerTime = time.time()
>>>>
>>>>      #create base filename at time of motion trigger
>>>>      fname = time.strftime("%Y-%m-%d_%H.%M.%S.h264",
>>>> time.localtime(triggerTime))
>>>>
>>>>      RAMname = os.path.join("/run/shm", fname)
>>>>      NASname = os.path.join("/mnt/CCTV/PiZero" , fname)
>>>>
>>>>      camera.start_preview()
>>>>
>>>>      #capture video to RAM device first
>>>>      camera.start_recording(RAMname)
>>>
>>>     I suggested trying NASname in that call, to directly capture to
>>> the end
>>> device rather than buffering in RAM first and then copying. If that
>>> fails,
>>> then the problem is not inherent to the script but is something wrong in
>>> the environment. RAMname (and later the shutil copyfile) was proposed to
>>> let the RPi capture to RAM without the overhead of network traffic
>>> (given
>>> that the network interface of an RPi is via the USB system)
>>>
>>>>      camera.wait_recording(10)
>>>>      camera.stop_recording()
>>>>      camera.stop_preview()
>>>>
>>>>      #copy from RAM device to NAS device
>>>>      #shutil.copyfile(RAMname, NASname)
>>>>      src = join('/', RAMname) #new lines here < I added this
>>>>      dst = join('/', NASname) < and this
>>>
>>>     Similarly, these do nothing useful since "src" and "dst" are never
>>> referenced in the code... And what they are doing is just trying to add
>>> another / to the front of the names generated earlier. But since / is
>>> the
>>> path separator in Linux, and .join() puts the components together by
>>> putting a path separator between them...
>>
>> Ok , I've removed them
>>>
>>> -=-=-=-=-
>>>>>> import os.path
>>>>>> import time
>>>>>> fname = time.strftime("%Y-%m-%d_%H.%M.%S.h264", time.localtime())
>>>>>> fname
>>> '2018-06-16_08.45.24.h264'
>>>>>> nname = os.path.join("/mnt/CCTV/PiZero", fname)
>>>>>> nname
>>> '/mnt/CCTV/PiZero/2018-06-16_08.45.24.h264'
>>>>>> dst = os.path.join("/", nname)
>>>>>> dst
>>> '/mnt/CCTV/PiZero/2018-06-16_08.45.24.h264'
>>>>>> dst == nname
>>> True
>>>>>>
>>> -=-=-=-=-
>>>
>>> ... there is no effect
>>>
>>>>
>>>>
>>>>      #delete file from RAM device
>>>>      os.unlink(RAMname)      #os.remove() is same function
>>>
>>>     I also asked you to comment that out... Since if the capture is
>>> going
>>> to NASname, there is no file in RAMname to be deleted
>>
>> Yes I did do that last night, but there was some sort of error. I
>> can't recall it now as it was late.
>>>
>>>>
>>>>     #set up next check time
>>>>      time.sleep((triggerTime + 20.0) - time.time())
>>>>
>>>
>>>     While picking up that beginners book on Linux, pick up a book on
>>> Python
>>> also...
>>>
>>>
>>
>> I have now commented out what you asked me to and also:
>>
>> camera.start_recording(NASname)
>>
>> The script is running without errors, and producing video files of
>> around 5 or 6MiB on the PiZero, but nothing of todays date on the NAS
>> box.
>> So the script is producing the files but not moving/copying them
>> accross to the NAS box
>
> No the script didn't run very long, as it produced this error which I
> now recall from last night.
>
> pi@raspberrypi:~/Downloads $ python intruder.py
> Traceback (most recent call last):
>    File "intruder.py", line 36, in 
>      camera.start_recording(NASname)
>    File "/usr/lib/python2.7/dist-packages/picamera/camera.py", line
> 1049, in start_recording
>      encoder.start(output, options.get('motion_output'))
>    File "/usr/lib/python2.7/dist-packages/picamera/encoders.py", line
> 812, in start
>      super(PiVideoEncoder, self).start(output)
>    File "/usr/lib/python2.7/dist-packages/picamera/encoders.py", line
> 375, in start
>      self._open_output(output)
>    File "/usr/lib/python2.7/dist-packages/picamera/encoders.py", line
> 331, in _open_output
>      self.outputs[key] = mo.open_stream(output)
>    File "/usr/lib/python2.7/dist-packages/picamera/mmalobj.py", line
> 344, in open_stream
>      stream = io.open(stream, 'wb' if output else 'rb', buffering)
> IOError: [Errno 13] Permission denied:
> '/mnt/CCTV/PiZero/2018-06-16_15.03.02.h264'
>
>
> I have just checked all the mounted directories, and they are all owned
> by pi:pi and have appropriate permissions.

Adendum:

The script as it is now runs for about 10 seconds and falls down with a
permission error.
It also produces 0 byte files on the NAS box, but no files on the PiZero
now.

--- SoupGate-Win32 v1.05