Is there a better way to do this? BASH

debugcn 投稿 Dev

Matthew

ssh -p 10551 -S /tmp/manpagemaker/ssh.connect -MfN [email protected]

for xe in ${XECOMMANDS[@]} ;do

    xe=$(echo ${xe#*-}) ; xe=$(echo ${xe%.*})
    ssh -p 10551 -S /tmp/manpagemaker/ssh.connect [email protected] t 'bash -s' < ./manpagemaker.sh "$xe"

done

Is there a better way to do this? I feel like my method is very inefficient.

I have wrote a script that builds a asciidoc template for a manpage. I'm writing manpages for the xe command that is part of the xapi. The "manpagemaker.sh" script depends on the xe command to execute properly, which is present on my remote xenserver cloud.

So instead of opening a new ssh connection for every instance of manpagemaker I need to run, could I some how use a named pipe and stream it do my xenserver cloud? Keep in my mind I need to add a redirect eventually to capture all the text sent to stdout.

Aleks-Daniel Jakimenko-A.

You can try piping a function to xargs:

showCommands() {
    for xe in ${XECOMMANDS[@]}; do
        xe=$(echo ${xe#*-}) ; xe=$(echo ${xe%.*})
        echo "$xe"
    done
}

showCommands | xargs -l1 ssh -p 10551 -S /tmp/manpagemaker/ssh.connect [email protected] t 'bash -s' < ./manpagemaker.sh

I don't know the contents of XECOMMANDS, but is seems like that this code is broken with cases when filenames contain space characters. Here is a way to fix that:

showCommands() {
    for xe in "${XECOMMANDS[@]}";do
        echo -n "$xe"
        echo -ne '\0'
    done
}

showCommands | xargs -0 -l1 ssh -p 10551 -S /tmp/manpagemaker/ssh.connect [email protected] t 'bash -s' < ./manpagemaker.sh

-l1 means that xargs will read the pipe input line by line.

Since filenames can't contain a null character I used \0 to split parameters.

If you want to pass more than one parameter to ssh, then you have to increase -l by one and add another echo output. Here's an example:

showCommands() {
    for xe in "${XECOMMANDS[@]}";do
        echo -n "$xe"
        echo -ne '\0'
        echo -n 'This is another parameter'
        echo -ne '\0'
    done
}

showCommands | xargs -0 -l2 ssh -p 10551 -S /tmp/manpagemaker/ssh.connect [email protected] t 'bash -s' < ./manpagemaker.sh

この記事はインターネットから収集されたものであり、転載の際にはソースを示してください。

侵害の場合は、連絡してください[email protected]

編集2021-06-23

コメントを追加

サインイン

分類Dev

Related 関連記事

記事

Is there a better way to do this? BASH

Is there a better way to do this? BASH

better way to do this in MATLAB?

Is there a better way to do Insertion Sort?

Is there a better way to do this than echo?

Do you know a better way to write this method?

Is there a better way to do this code? Perhaps using filter or reduce?

Is there a better way of comparing dates

A better way to detect the cursor?

Better way to return boolean

A better way to introspect a capture

Is there a better way to loop code?

Is there a better way to do modulo in a finite field when directly working on polynomials rather than binary numbers?

Is there a better way to define a global variable?

Decorator with Arguments: Would this be a better way?

Is this possible using LEAD or is there a better way?

Is there a way of making discord embeds better?

Better way to override a function definition

Is there a better way to divide this string into substrings?

In bash, is it generally better to use process substitution or pipelines

Are these two bash snippets equivalent and if not which is better?

Is there a better way to re-use plots Matplotlib?

Better way of capturing multiple same tags?

Better way to execute nested for loops to generate a dict?

Better way of error handling in Kafka Consumer

rxJava better way to wait on a specific sequence of values

Which way of setting fields value is better and why?

Which way of setting fields value is better and why?

AngularJS: Is there a better way to achieve this than the one specified?

Better way to write jquery add/remove class

better way to load 2 dropdown in mvc