Bug 34187

Summary:

Учитывает регистр

Product:

Sisyphus

Reporter:

Evgenii Terechkov <evg>

Component:

apf

Assignee:

Gremlin from Kremlin <gremlin>

Status:

CLOSED FIXED

QA Contact:

qa-sisyphus

Severity:

normal

Priority:

CC:

gremlin, mcpain

Version:

unstable

Hardware:

all

OS:

Linux

Attachments:

Description	Flags
Apply case-insensitive search	none
Case-insensitive search for packages and files	none

Description Evgenii Terechkov 2017-11-13 18:53:14 MSK

=8<===========================================================
evg@thinkpad ~ $apf search BV9ARM
evg@thinkpad ~ $apf search Bv9ARM
bind-doc
evg@thinkpad ~ $
=8<===========================================================

надо чтобы search не учитывал регистр.

Comment 1 Олег Соловьев 2017-11-14 09:43:44 MSK

Created attachment 7269 [details]
Apply case-insensitive search

Comment 2 Evgenii Terechkov 2017-11-14 10:11:31 MSK

Кроме этого патча, возможно, есть и другие места в коде где проверку лучше заменить на регистронезависимую.

Comment 3 Олег Соловьев 2017-11-14 10:36:40 MSK

Created attachment 7270 [details]
Case-insensitive search for packages and files

Comment 4 Evgenii Terechkov 2017-11-14 18:29:59 MSK

Вообще не понимаю логику:
=8<==================================================================
evg@thinkpad ~ $apf search bin/pv
python-module-pyramid
python3-module-pyramid
pve-common
lvm2
pve-manager
pve-cluster
mgetty-voice
sundials-examples
pvm
pvm-devel
pve-firewall
pve-ha-manager
pve-container
pve-storage
pve-access-control
Задание "apf search bin/pv" заняло 20,685
evg@thinkpad ~ $apf search 'bin/pv\b'
pv
Задание "apf search 'bin/pv\b'" заняло 20,172
evg@thinkpad ~ $
=8<==================================================================

почему bin/pv не нашлось, а с границей слова \b - нашлось? Если заключить bin/pv в кавычки то тоже находится.

Comment 5 Олег Соловьев 2017-11-15 11:48:50 MSK

> почему bin/pv не нашлось, а с границей слова \b - нашлось? Если заключить
> bin/pv в кавычки то тоже находится.

Вы используете в паттерне символ "/", который по умолчанию используется как разделитель самого паттерна и флагов и perl выполняет "/bin/pv/i" и матчит строку "bin", потому что он не понимает, что "/pv" - тоже часть строки.

PS после применения патча на case-insensitive
[mcpain@McPain-Work apf]$ apf search bin/pv
lvm2
pv
mgetty-voice
sundials-examples
pvm
pvm-devel
python-module-pyramid
python3-module-pyramid
pve-common
pve-manager
pve-cluster
pve-firewall
pve-ha-manager
pve-container
pve-storage
pve-access-control

Comment 6 Evgenii Terechkov 2017-11-20 04:00:48 MSK

ping?

Comment 7 Repository Robot 2017-11-20 09:23:59 MSK

apf-0.3-alt1 -> sisyphus:

Tue Nov 14 2017 Oleg Solovyov <mcpain@altlinux> 0.3-alt1
- search is now case-insensitive (Closes: #34187)

Wed Sep 20 2017 Gremlin from Kremlin <gremlin@altlinux> 0.2-alt1
- incorporated fixes proposed by mcpain@
- check whether cache directory is writable
- try to create cache directory if it does not exist
- stop update process on rsync failure or interrupt