Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Case sensitive search with SEARCH_TYPE_PCRE #2764

Closed
mc-butler opened this issue Mar 27, 2012 · 8 comments
Closed

Case sensitive search with SEARCH_TYPE_PCRE #2764

mc-butler opened this issue Mar 27, 2012 · 8 comments
Assignees
Labels
area: search Search subsystem prio: medium Has the potential to affect progress
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2764
Reporter vokac (vokac@….cvut.cz)

When mc is compiled with SEARCH_TYPE_PCRE (and without SEARCH_TYPE_GLIB, e.g. on systems with old glib < 2.14) case (in)sensitive search works in opposite - see attached patch.

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by vokac (vokac@….cvut.cz) on Mar 27, 2012 at 15:36 UTC

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Apr 5, 2012 at 12:57 UTC (comment 1)

  • Owner set to slavazanko
  • Status changed from new to accepted

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Apr 5, 2012 at 13:11 UTC (comment 2)

  • Votes set to slavazanko
  • Milestone changed from Future Releases to 4.8.3
  • Keywords changed from case sensitive, pcre to case sensitive, pcre, stable-candidate
  • Branch state changed from no branch to on review

created branch 2764_pcre_case_sentitive [0c9764ca237a3413bb8101525b6bcb9da18e5d59]
Review, please.

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Apr 5, 2012 at 13:18 UTC (comment 3)

  • Votes changed from slavazanko to slavazanko angel_il

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Apr 5, 2012 at 13:18 UTC (comment 4)

  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 5, 2012 at 13:27 UTC (comment 5)

  • Keywords changed from case sensitive, pcre, stable-candidate to stable-candidate

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Apr 9, 2012 at 8:33 UTC (comment 6)

  • Status changed from accepted to testing
  • Resolution set to fixed
  • Branch state changed from approved to merged
  • Votes changed from slavazanko angel_il to committed-master

merged to master:

git log --pretty=oneline e63fc08..8c0621b

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Apr 9, 2012 at 8:36 UTC (comment 7)

  • Status changed from testing to closed
  • Votes changed from committed-master to committed-master committed-stable
  • Keywords stable-candidate deleted

merged to stable:

git log --pretty=oneline 22785c8..a99d2b2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: search Search subsystem prio: medium Has the potential to affect progress
Development

No branches or pull requests

2 participants