Dotaz: SED substituce s $i - nefunkcni

[Filip Huska]

Dobry den,

jde mi o script do cronu, kde kazda skupina web* bude mit uzivatele www-data. Nejde mi o jina reseni jako ulozit do druheho souboru a diffnout, ci podobne, jde mi o reseni ciste, tzn. presne tohle ... ovsem sed mi nebere neznamou. Jak udelat, aby ji sed bral ?

#!/bin/sh

for i in `grep web* /home/rexxor/gransy/group | grep -v www-data` ;do

sed -e s/$i/$i,www-data/ /home/rexxor/gransy/group

done

Odpovědi

[Zdeněk Burda]

Ahoj,

tak trochu jsem nepochopil co je cílem dané oprace (jak vypadají vstupní data, jak má vypadat výstup), ale to je jedno...

Když nepoužiješ cron, tak ti ten skriptík funguje?

Co to hlásí za chybu? Třeba ten skript pracuje úplně správně, jen ty očekvýš, že bude dělat něco jiného

[Filip Huska]

Zkusim to rozebrat ..

Vyber vsechny skupiny zacinajici web ze souboru group a neobsahujici souslovi www-data (prvni radek for).

Pro vsechny tyto vybrane skupiny pridej na konec "www-data" (prirad uzivatele)a zapis je zpet do group.

Problem : sed -e s/$i ...

Nenacita se neznama z i, ale $i se nacita jako string.

[Zdeněk Burda]

Nebylo by lepší vše udělat přímo sedem místo toho for cyklu?

sed -e s/... určitě nic do souboru nezapíše, vypíše to na standarní výstup.

[Filip Huska]

Nechete si ten script nejdrive odzkouset bez promenych ?

Ja bych nerad byl nezdvorily, ale zpochybnenim problemu dany problem nevyresite a script je bez promenych zcela funkcni. Jde mi o to, inkludovat cistou formou $i do sedu, coz je pochopitelne.

Uvedeny priklad je osekany jen jak je to mozne, aby neztratil smysl a nemusel jsem nikoho zahlcovat zbytecnymy podrobnostmi.

Muj problem zni jasne : substituce v sedu s promenou $i na $i,www-data
Vstup - soubor s daty :

web1...........cricket

web2...........cricket

web3...........www-data,cricket

Vystup - ten samy soubor :

web1 ..........cricket,www-data

web2...........cricket,www-data

web3...........www-data,cricket

[Zdeněk Burda]

Nevím proč bych si měl skrip zkoušet bez proměných, podle mého ten skript funguje a dělá přesně to co má:

[tsunami@vodik]$ cat /tmp/a
web1:cricket
web2:cricket
web3:www-data,cricket


[tsunami@vodik]$ for i in `grep web* /tmp/a | grep -v www-data`; do sed -e s/$i/$i,www-data/ /tmp/a ; done
web1:cricket,www-data
web2:cricket
web3:www-data,cricket
web1:cricket
web2:cricket,www-data
web3:www-data,cricket

Jak jsem psal, ten sed nemodifikuje soubor, ale vypisuje to na standardní výstup.

[Zdeněk Burda]

Předpokládám, že tohle bude třeba lepší (bez for cyklu, ale je to s dočasným souborem):

[tsunami@vodik]$ cat /tmp/a
web1:cricket
web2:cricket
web3:www-data,cricket

[tsunami@vodik]$ sed -e /www-data/\!s/^web.*/\&,www-data/ /tmp/a >/tmp/tmp_soubor.$$; mv /tmp/sed_temp_soubor.$$ /tmp/a

[tsunami@vodik]$ cat /tmp/a
web1:cricket,www-data
web2:cricket,www-data
web3:www-data,cricket

[Dunric]

Ta proměnná i se určitě expanduje.

Nevrací ti ten grep náhodou hodnotu např. obsahující mezeru nebo lomítko ?

Minimálně bych kvůli mezerám uzavřel výraz pro sed do uvozovek.

sed -e "s/$i/$i,www-data/" /home/rexxor/gransy/group

[Dunric]

Ještě jsem si všiml následující chyby, která může být příčinou problému.

for i in `grep web* /home/rexxor/gransy/group | grep -v www-data` ;do

Protože je grepu předáván vzorek obsahující hvězdičku bez uvozovek, shell se ji pokusí expandovat a pokud v adresáři, ze kterého se spouští, existuje(í) soubory začínajicí znaky web, jejich název se dosadí do vzorku.

[Michal Marek (twofish)]

A pokud ji shell neexpanduje, tak je tam naprosto zbytečně (leda že by tazatel chtěl hledat řetězec 'we').

[Zdeněk Burda]

Hezký, tak sem se ten for cyklus a grep snažil ignorovat, že jsem si ani nevšim téhle chyby

Každopádně to nic nemění na tom, že je ten for zbytečný, protože procházet sedem pro každý nalezený řetězec celý soubor je nesmysl. Lepší je vše nahradit najednou.

[Filip Huska]

Velice Vam dekuji vsem za moznosti ... vyreseno