fopetesl

Command Prompt batch file fails

39 posts in this topic

#1 ·  Posted (edited)

I have several batch files, e.g.

echo off
    echo !DT | ncat 192.168.1.6 80 > scan1.dta

This data1.bat works as expected by saving data from !DT command.

I have tried several variations from AutoIt

;  $DOS = RunWait(@ComSpec & " /k " & "ncat 192.168.1.6 80 <getdata >scan1.dta", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)
;  $DOS = Run(@ComSpec & " /k " & "ncat 192.168.1.6 80 <getdata >scan1.dta", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)
;  $DOS = Run(@ComSpec & " /c " & "echo ?DT | ncat 192.168.1.6 80 > scan1.dta & @CRLF", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)
  $DOS = Run(@ComSpec & " /c " & "data1.bat & @CRLF", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)

but none save the data to scan1.dta

Run out of ideas so far...

 

Edited by fopetesl
Missing code

The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites



Are you sure you are looking in the right subdirectory ?
Why do you have the @CRLF in there ? You don't need to give a Enter when running a script!

Run it as test like this and see what directory it is in and whether there is an error:

$DOS = Run(@ComSpec & " /k " & "data1.bat", "", default, $STDERR_CHILD + $STDOUT_CHILD)

Jos


Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

No change, Jos.

Not sure what you mean by "see what directory it is in" since although the command prompt windows flashes up it isn't long enough to take note.

I can see that the scan1.dta files has been opened since the timestamp has been updated but it contains no data.

I also did a search of the whole drive in case scan1.dta was placed somewhere else.

 


The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites

Still the same, Jos. syntax as you suggested:

  $DOS = Run(@ComSpec & " /k " & "data1.bat", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)


The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites

#6 ·  Posted (edited)

The /k should keep the command window open, does it? If so, which directory is it showing? "Still the same" is not answering the question that Jos has asked twice now.

Edited by JLogan3o13

√-1 2^3 ∑ π, and it was delicious!

Share this post


Link to post
Share on other sites

#7 ·  Posted (edited)

15 minutes ago, fopetesl said:

Still the same, Jos. syntax as you suggested:

  $DOS = Run(@ComSpec & " /k " & "data1.bat", "", @SW_HIDE, $STDERR_CHILD + $STDOUT_CHILD)

This is NOT the commandline I provide!  Check for the difference.

Jos

Edited by Jos

Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

#8 ·  Posted (edited)

Sorry. Too many balls in the air.

No. Windows does not stay open even with corrected syntax.

$DOS = Run(@ComSpec & " /k " & "data1.bat", "", default, $STDERR_CHILD + $STDOUT_CHILD)

Edited by fopetesl
add code

The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites
31 minutes ago, fopetesl said:

Sorry. Too many balls in the air.

No. Windows does not stay open even with corrected syntax.

 

$DOS = Run(@ComSpec & " /k " & "data1.bat", "", default, $STDERR_CHILD + $STDOUT_CHILD)

 

Well part of the problem that I believe that @Jos is trying to get to is that you are not specifying the directory in which the file is located.  The only case in which you wouldn't have to is if the script that you are running is in the same directory as the file you are trying to interact with.  If it is not, then that would be the issue: the script has no idea where the file is, because it is only looking in the current working directory (aka the directory in which the script is located).  You need to specify where the file is located in the "working Directory parameter.  Using the code you provided:

$DOS = Run(@ComSpec & " /k " & "data1.bat", "Path\To\Folder\Containing\file", default, $STDERR_CHILD + $STDOUT_CHILD)

Share this post


Link to post
Share on other sites

The simple point I am getting to is that I think the CMD prompt should stay open when /k is used so you can see any error and the workdir it is in.

When that is not the case try an even simpler version to understand some of the basics:

$DOS = Run(@ComSpec & " /k " & "data1.bat")

Jos


Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

Is there any reason, other than having to rewrite them, that you can't get rid of the batch files entirely and just do everything with AutoIt?


If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Share this post


Link to post
Share on other sites

A quick test revealed indeed that nothing was shown with the STDERR and STDOUT parameters. These 2 give the same result and show me an error message.
The first one as console output and the second in the CMD prompt:

#include <AutoItConstants.au3>
$DOS = Run(@ComSpec & " /k " & "data1.bat", "", default, $STDERR_CHILD + $STDOUT_CHILD)
; Wait until the process has closed using the PID returned by Run.
ProcessWaitClose($DOS)
; Read the Stdout stream of the PID returned by Run. This can also be done in a while loop. Look at the example for StderrRead.
ConsoleWrite('@@ Debug(' & @ScriptLineNumber & ') : StdoutRead($iPID) = ' & StdoutRead($DOS) & @CRLF ) ;### Debug Console
ConsoleWrite('@@ Debug(' & @ScriptLineNumber & ') : StdoutRead($iPID) = ' & StderrRead($DOS) & @CRLF ) ;### Debug Console
;
$DOS2 = Run(@ComSpec & " /k " & "data1.bat")

Jos

1 person likes this

Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

MattHiggs, I tried your suggestion some while back which didn't resolve the issue.

note also

Global $BatchDir = "C:\ncat\"

is at top of file.

BrewMan. I'd love to include these in my script but my tiny brain can't figure out how to pipe commands to ncat.exe and collect any return data.  Oddly though some .bat files do work and send a response I'd expect, e.g.

Run(@ComSpec & " /k " & "IsHome.bat", "", @SW_HIDE)
         Sleep(400)
         Local $hFileOpen = FileOpen("homerep", $FO_BINARY)
         Local $sFileRead = FileRead($hFileOpen)
         FileClose($hFileOpen)
         Local $sStr = StringTrimLeft(StringToBinary("?HMO"), 4)
         If StringInStr($sFileRead, $sStr) Then
           MsgBox(0, "", "Head Home", 2)
           return(0)
         EndIf 

This just checks the (same) machine earlier in the script for scanning head at 'home' position.

The only difference I can see between the working script and the failure is the amount of data returned.  "?HMO" returns eight characters, data1.bat returns 754 hex bytes which it does from command prompt but just creates an empty file from AutoIt.

Jos, I'll get to your read output/errors in a while, (I hope), still have balls in the air.

 


The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites
9 hours ago, fopetesl said:

MattHiggs, I tried your suggestion some while back which didn't resolve the issue.

note also

Global $BatchDir = "C:\ncat\"

is at top of file.

BrewMan. I'd love to include these in my script but my tiny brain can't figure out how to pipe commands to ncat.exe and collect any return data.  Oddly though some .bat files do work and send a response I'd expect, e.g.

 

Run(@ComSpec & " /k " & "IsHome.bat", "", @SW_HIDE)
         Sleep(400)
         Local $hFileOpen = FileOpen("homerep", $FO_BINARY)
         Local $sFileRead = FileRead($hFileOpen)
         FileClose($hFileOpen)
         Local $sStr = StringTrimLeft(StringToBinary("?HMO"), 4)
         If StringInStr($sFileRead, $sStr) Then
           MsgBox(0, "", "Head Home", 2)
           return(0)
         EndIf

This just checks the (same) machine earlier in the script for scanning head at 'home' position.

The only difference I can see between the working script and the failure is the amount of data returned.  "?HMO" returns eight characters, data1.bat returns 754 hex bytes which it does from command prompt but just creates an empty file from AutoIt.

Jos, I'll get to your read output/errors in a while, (I hope), still have balls in the air.

 

So are you setting $batchdir as the working directory in the run function?

$DOS = Run(@ComSpec & " /k " & "data1.bat", $batchdir, default, $STDERR_CHILD + $STDOUT_CHILD)

If this is the case, you need to get rid of the last "\" so that $batchdir = "C:\ncat"

when you put the last "\" at the end of the directory path, the variable no longer equals the parent directory.

Share this post


Link to post
Share on other sites

MattHiggs. VERY carefully followed your suggestions.

No change. Still have empty scan1.dta but file timestamp is updated to current.

Same applies to RunWait() except it takes several seconds before Command Prompt window flashes very briefly before returning to GUI Tab.

So the .dta file is clearly opened and closed without having data written.

What is the difference between Run() and ShellExecute()?

 


The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites
3 hours ago, fopetesl said:

What is the difference between Run() and ShellExecute()?

ShellExecute() runs a program with its default interpreter. For example executing a word document would automatically open Microsoft Word. Run is usually used to run more specific programs with an exact path. They are very similar but can have slightly different use cases. I've also heard that Run is generally faster because it doesn't have to go through a separate API. If you want some more information I found this thread that explains it some more.


UHJvZmVzc2lvbmFsIENvbXB1dGVyZXI=

Share this post


Link to post
Share on other sites
On 2017-6-5 at 5:35 PM, fopetesl said:

Jos, I'll get to your read output/errors in a while, (I hope), still have balls in the air.

....  and? 


Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

#18 ·  Posted (edited)

On 05/06/2017 at 3:25 PM, Jos said:

A quick test revealed indeed that nothing was shown with the STDERR and STDOUT parameters. These 2 give the same result and show me an error message.
The first one as console output and the second in the CMD prompt:

#include <AutoItConstants.au3>
$DOS = Run(@ComSpec & " /k " & "data1.bat", "", default, $STDERR_CHILD + $STDOUT_CHILD)
; Wait until the process has closed using the PID returned by Run.
ProcessWaitClose($DOS)
; Read the Stdout stream of the PID returned by Run. This can also be done in a while loop. Look at the example for StderrRead.
ConsoleWrite('@@ Debug(' & @ScriptLineNumber & ') : StdoutRead($iPID) = ' & StdoutRead($DOS) & @CRLF ) ;### Debug Console
ConsoleWrite('@@ Debug(' & @ScriptLineNumber & ') : StdoutRead($iPID) = ' & StderrRead($DOS) & @CRLF ) ;### Debug Console
;
$DOS2 = Run(@ComSpec & " /k " & "data1.bat")

Jos

Jos, not sure what I actually see

>"C:\Program Files (x86)\AutoIt3\SciTE\..\autoit3.exe" /ErrorStdOut "C:\S3_GUI\HauterS3 with_Tabs1.au3"    
@@ Debug(211) : StdoutRead($iPID) =
C:\ncat>echo off
    C:\ncat>
@@ Debug(212) : StdoutRead($iPID) = Ncat: .
    >Exit code: 0    Time: 34.01

It's clear that the 2nd Run() $DOS2 actually does what it's supposed to do and writes the correct data to scan1.dta.  If I comment this line out then no data appears.  Also I now get a Command Prompt window opened. It takes approx. eight seconds for ProcessWait() to terminate.

Edit: Tried "ShellExecute("data1.bat")" .. got a Command Prompt window for about eight seconds which closed without error but still no data saved. The batch file works perfectly from Command Prompt.

So there seems to be a bug in AutoIt when running some Command Line programs?

 

Edited by fopetesl

The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites
6 hours ago, fopetesl said:

It's clear that the 2nd Run() $DOS2 actually does what it's supposed to do and writes the correct data to scan1.dta.

Then use that format and don't tell AutoIt3 to capture the STDOUT & STDERR since you want to pipe that into a file anyways, or do you see any issues with that?

Jos


Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

#20 ·  Posted (edited)

16 hours ago, Jos said:

Then use that format and don't tell AutoIt3 to capture the STDOUT & STDERR since you want to pipe that into a file anyways, or do you see any issues with that?

Jos

I've no issue except the second (not $DOS2) call to capture data to scan2.dta always fails even if I call it twice

$DOS2 = Run(@ComSpec & " /k " & "data1.bat") ; 2nd call works here
Sleep(100)
$DOS3 = Run(@ComSpec & " /k " & "data2.bat")
Sleep(100)
$DOS3 = Run(@ComSpec & " /k " & "data2.bat") ; 2nd call here doesn't
Edited by fopetesl
Missing text

The most powerful number in the Universe.  Zero.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now

  • Similar Content

    • rootx
      By rootx
      Hi, I need to convert this batch, I follow microsoft specs here https://technet.microsoft.com/en-us/library/bb491005.aspx
      Ex:
      start /b 1.exe "path\2.exe"
      start /b mypgrcommand
      start /b /wait 3.exe "path\4_.exe"
      taskkill /F /IM 5.exe
      Somethings like this...
      RunWait( @COMSPEC & " /c start /b 1.exe 'path\2.exe' start /b mypgrcommand start /b /wait 3.exe 'path\4_.exe' taskkill /F /IM 5.exe", "", @SW_SHOW ) Thx
    • TheSaint
      By TheSaint
      TeraCopy Timer - A program of mine, front end really, that I have been working on (on & off) for some time, but not shared here before, that I recall. Third party program TeraCopy is required.
      Many of you will be familiar with TeraCopy, and it has been discussed here at AF on several occasions, in various sub forums, including Chat.
      BE ADVISED - I still use an older version of TeraCopy (v2.27), a goody but an oldy, so have only tested my program with that. It may work with the newer v3.xx, or may be easily adjusted to do so, but I have not yet gone that route. Likewise, I only use it with Win 7 and Win XP, but I don't imagine there should be any issues with newer Windows. Add to that, my program being devised and created in AutoIt v3.3.0.0.
      So what is TeraCopy Timer and why might you need it?
      I devised my program for two main reasons really, based on giving me greater control, using the Free version of TeraCopy, which has some limitations (perhaps even with the paid full version). My program utilizes the features of the TeraCopy command-line.
      1. I was fed up with Thumbs.db files regularly halting the process in Win XP or preventing a folder being deleted after a move. Only happens with Win XP.
      2. I wanted to automate delays between jobs, allowing all HDD's to rest periodically when doing large and lengthy jobs. Letting HDD's heat up too much, can have tragic results or considerably shorten their lifespan. Prevention is better than cure or just relying on monitoring software etc.
      --------------------------------------------------------------------------------------
      Those familiar with TeraCopy use, know you can run multiple copy or move jobs either simultaneously or concurrently (one immediately after another has finished). With the free version at least, you have no control over precise order or timing, and no way to deal (without manual interaction) with the pesky Thumbs.db files in Win XP. TeraCopy Timer allows you to do all those things, and in a batch fashion.
      --------------------------------------------------------------------------------------
      Some screenshots and brief explanation
      Main Window

      You can browse to set the Source and Destination paths, or like me, just use Drag & Drop to those inputs.
      Once the Destination path has been set, the MIN (minimalist GUI) button becomes available. In Minimal mode, you get a further level of automation, once initial options are set, and thus less prompts ... none in fact, as jobs are created automatically based on either COPY or MOVE.
      Minimalist Window

      In the Minimalist mode, you can only use Drag & Drop to add a source file or folder, and COPY or MOVE is permanently set when the window first opens, via a choice prompt.
      Assigning a WAIT is simple and easy, and is the latest feature added to the program ... I used a much more complex variant before that (see Advanced Delay).

      The program now displays three file size reports.
      Individual Size of the current (last added) job (Blue label).
      Total Size of all jobs combined (Black label).
      Subtotal Size for each grouping of jobs, defined by a WAIT selection (Red label).
      You toggle between Black and Red, by just clicking that label. If no WAIT has been set, then the values will be the same.
      The wait of 5 minutes in the screenshot above is set for the source shown. It means wait 5 minutes before copying (or moving) that source.
      More sources added from that point, add to a new subtotal. To see the previous subtotal, you need to be at the main (MAX) window, and select the prior job.
      So every time a WAIT is clicked for a source a new subtotal count is started.

      In the screenshot above, you can see the Job name, and that it is Job number 5 order wise. MOVE has been set for that job.
      Selecting Job 4, you will see the previous (complete) Subtotal. Selecting job 3 (in this instance), you would see the subtotal up to that job.
      Click the red Total label and it will change to Black, and show you the total size over all, as shown in the second screenshot above.
      This new WAIT feature is the simplest and best way to use the program generally (in my view) ... but check out the following, as it is not always the case.
      Advanced Delay Window

      The Advanced Delay Options, are a further level of automation, added during the early stages of development, before I thought to create the Minimalist window and show Sizes. It was before I decided to put a lot more effort into the program. As with all my programs though, it is continual use that eventually dictates what I ultimately want to happen, to make my life easier ... simpler, better, smarter, quicker.
      The main difference between this older method, and the new WAIT one, where you specify delays precisely, is that the program attempts to determine the best moments to pause, based on various factors, which you setup and can vary between different types of Job sessions.
      The chief purpose for all the advanced options, is an attempt to cater for the difference between moving a small number of big files and a lot of small files. Moving or Copying lots of small files (hundreds or thousands), as many would know, can heat up a HDD far quicker and to a much higher temperature than a small number of (even very) large files. I guess that is primarily due to the sheer number of indexes that need to be created, and with small files are done at a staggering rate.
      Which method you use, is up to you, and should be governed by the type of job. Both methods can be used together, but not advised if you don't want the possibility of unnecessary extra long delays. EDIT - That said, you could use the WAIT option just for a delayed start of the first job ... perhaps your PC is busy doing something else until then, but you want to go and do something else for a bit, and have it all done by the time you return (i.e. watch a movie).
      WARNING - It should go without saying, that you take all the usual precautions. For instance, don't use move with the only copy you have of precious files. BACKUP BACKUP BACKUP! Don't hold me liable for anything, except for being a nice guy who likes to share.
      TeraCopy Timer v2.0.zip
      Enjoy!
      P.S. I am not affiliated in any way with those who created and provide the excellent third party program - TeraCopy.
    • Eggsplorer
      By Eggsplorer
      Hi,
      Ich hab ein Script für Windows Updates, dass nach dem OOBE aufgerufen werden soll. Run/RunOnce ist daher nicht möglich.
      Stattdessen möchte ich das Script in den Autostart schreiben lasse. Wenn die OOBE beendet wird und der Administrator angemeldet wird, soll das Update-Script einmalig aufgerufen werden und sich anschl. selbst löschen.
      Lang rede, kurzer Sinn:
      Wie kann ich folgendes Script in den Autostart schreiben?
      RunWait(@ComSpec & " /c " & "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -File C:\UA\Scripts\win-updates.ps1 -MaxUpdatesPerCycle 100 -RestartRequired 1", "") Ich würde ungern eine BAT oder EXE reinkopieren, da ich es unsauber finde. Die müsste ja auch zusätzlich irgendwo liegen.
      Danke
      Eggsplorer
    • ur
      By ur
      I am running the tomcat batch file through autoit   and need to wait until the below line I get in console output.

      But when I am trying to read the console output using the ProcessEx UDF, I am getting only the partial output.
      So, can you please suggest how to handle this.
    • rinoatwisted
      By rinoatwisted
      Hello guys,
      im just started in programming and all..
      and i wanted to learn autoscript to.
      Was wondering if any can convert my  one-liner batch script to an autoit script -
      FOR /F %%i IN (names.txt) DO @copy system.act %%i.act
       
      So what this does is open and read the file "names.txt" incrementally, and copy a certain file named "system.act" and rename it to each name that is listed on "names.txt"
       
      So im trying this on autoit but no idea how to start file manipulation and do command -
       
      #include <File.au3>
      $file = "c:\names.txt"
      FileOpen($file, 0)
      For $i = 1 to _FileCountLines($file)
          $line = FileReadLine($file, $i)
       
      Can anyone help me or guide me on how to continue this to get the result that i want..:)?
      Thanks