picnoir/Systemd
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[PATCH] udev - activate formt length attribute

Browse source 
We carried the the old callout part selector syntax for two releases
now after it was replaced by the new %c{1} syntax. So here we remove
the old syntax and use the code to possibly specify the maximum count
of chars to insert into the string. It will work with all of our format
chars.
I don't know if somebody will use it, but the code is already there :)

's%3s{vendor}' returns "IBM" now, instead of "IBM-ESXS".

Also added is a test for it and a few words in the man page.
This commit is contained in:
kay.sievers@vrfy.org 2004-02-28 06:53:25 -08:00 committed by Greg KH
parent e408796e91
commit 63ead27c22
3 changed files with 22 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
namedev.c
View file

@ -220,7 +220,7 @@ static void apply_format(struct udevice *udev, char *string, size_t maxsize,
char *pos2;
char *pos3;
char *attr;
int num;
int len;
int i;
char c;
struct sysfs_attribute *tmpattr;
@ -232,7 +232,7 @@ static void apply_format(struct udevice *udev, char *string, size_t maxsize,
if (pos != NULL) {
pos[0] = '\0';
tail = pos+1;
num = get_format_len(&tail);
len = get_format_len(&tail);
c = tail[0];
strfieldcpy(temp, tail+1);
tail = temp;
@ -274,7 +274,7 @@ static void apply_format(struct udevice *udev, char *string, size_t maxsize,
if (strlen(udev->program_result) == 0)
break;
/* get part part of the result string */
i = num; /* num syntax is deprecated and will be removed */
i = 0;
if (attr != NULL)
i = atoi(attr);
if (i > 0) {
@ -317,6 +317,10 @@ static void apply_format(struct udevice *udev, char *string, size_t maxsize,
dbg("unknown substitution type '%%%c'", c);
break;
}
/* truncate to specified length */
if (len > 0)
pos[len] = '\0';
strnfieldcat(string, tail, maxsize);
}
}

9
test/udev-test.pl
View file

@ -191,6 +191,15 @@ EOF
expected => "Major:8:minor:3:kernelnumber:3:bus:0:0:0:0" ,
conf => <<EOF
BUS="scsi", PLACE="0:0:0:0", NAME="Major:%M:minor:%m:kernelnumber:%n:bus:%b"
EOF
},
{
desc => "test NAME substitution chars (with length limit)",
subsys => "block",
devpath => "block/sda/sda3",
expected => "M8-m3-n3-b0:0-sIBM" ,
conf => <<EOF
BUS="scsi", PLACE="0:0:0:0", NAME="M%M-m%m-n%n-b%3b-s%3s{vendor}"
EOF
},
{

8
udev.8
View file

@ -209,7 +209,7 @@ fields support simple printf-like string substitution:
.TP
.B %n
The "kernel number" of the device.
for example, 'sda3' has a "kernel number" of '3'
For example, 'sda3' has a "kernel number" of '3'.
.TP
.B %k
The "kernel name" for the device.
@ -239,7 +239,11 @@ my be selected by specifying the part number as a attribute:
The content of a sysfs attribute.
.TP
.B %%
The '%' char itself.
The '%' character itself.
.P
The count of charcters to insert may be limited by specifying
the format length value. For example, '%3s{file}' will only insert
the first three characters of the sysfs attribute.
.P
.RI "A sample " udev.rules " might look like this:"
.sp