Bug 34187 - Учитывает регистр
Summary: Учитывает регистр
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: apf (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Gremlin from Kremlin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-13 18:53 MSK by Evgenii Terechkov
Modified: 2017-11-20 09:23 MSK (History)
2 users (show)

See Also:


Attachments
Apply case-insensitive search (379 bytes, patch)
2017-11-14 09:43 MSK, Олег Соловьев
no flags Details | Diff
Case-insensitive search for packages and files (637 bytes, patch)
2017-11-14 10:36 MSK, Олег Соловьев
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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