Tails

#1 Updated by intrigeri 2017-06-14 07:40:58

• Status changed from Confirmed to In Progress
• % Done changed from 0 to 10
• Feature Branch set to feature/12687-lighter-vim

#2 Updated by intrigeri 2017-06-14 19:22:02

• Assignee changed from intrigeri to anonym
• % Done changed from 10 to 50
• QA Check set to Ready for QA

#3 Updated by intrigeri 2017-06-29 10:33:39

• blocks Feature #13234: Core work 2017Q3: Foundations Team added

#4 Updated by anonym 2017-08-14 15:04:34

• Assignee changed from anonym to intrigeri
• % Done changed from 50 to 70

So the vim-tiny package installs a binary called vim.tiny, not vim. The latter is probably what users expects, so we should symlink /usr/bin/vim -> /usr/bin/vim.tiny. I pushed a commit to the branch doing that; please merge if you are happy with it!

#5 Updated by intrigeri 2017-08-19 13:43:38

• Assignee changed from intrigeri to anonym
• QA Check changed from Ready for QA to Info Needed

I’m confused. This is supposed to be handled automatically with the alternatives system. In a pbuilder Stretch chroot:

root@ensifera:/# apt install vim-tiny
[…]
Setting up vim-tiny (2:8.0.0197-4) ...
root@ensifera:/# ls /etc/alternatives/vim*
/etc/alternatives/vim  /etc/alternatives/vimdiff
root@localhost:/# ls -l /etc/alternatives/vim*
lrwxrwxrwx 1 root root 18 Jul  3  2016 /etc/alternatives/vim -> /usr/bin/vim.basic
lrwxrwxrwx 1 root root 18 Jul  3  2016 /etc/alternatives/vimdiff -> /usr/bin/vim.basic
root@localhost:/# ls -l /usr/bin/vim*
lrwxrwxrwx 1 root root      21 Jul  3  2016 /usr/bin/vim -> /etc/alternatives/vim
-rwxr-xr-x 1 root root 2413032 Apr 23 12:10 /usr/bin/vim.basic
-rwxr-xr-x 1 root root 1062744 Apr 23 12:10 /usr/bin/vim.tiny
lrwxrwxrwx 1 root root      25 Jul  3  2016 /usr/bin/vimdiff -> /etc/alternatives/vimdiff
-rwxr-xr-x 1 root root    2099 Apr 23 12:10 /usr/bin/vimtutor

So just to clarify: did you actually notice on the proposed branch the problem you’re trying to solve, or were you merely guessing there might be a problem?

#7 Updated by intrigeri 2017-08-23 10:01:15

• Assignee changed from anonym to intrigeri

#8 Updated by anonym 2017-08-24 23:10:38

• QA Check changed from Info Needed to Dev Needed

intrigeri wrote:
> I’m confused. This is supposed to be handled automatically with the alternatives system. In a pbuilder Stretch chroot:
>
> […]

Well, good!

> So just to clarify: did you actually notice on the proposed branch the problem you’re trying to solve, or were you merely guessing there might be a problem?

Sorry for my crappy report above! I noticed it when I tried to start vim but it didn’t exist. I actually also checked update-alternatives --list vim which said that there were no alternatives (and I just re-checked to confirm, and then I checked a bit more and e.g. vi uses the vim.tiny alternative). So there is a real problem.

#9 Updated by intrigeri 2017-08-25 14:16:23

• Assignee changed from intrigeri to anonym
• % Done changed from 70 to 80
• QA Check changed from Dev Needed to Ready for QA

Fixed the problem with update-alternatives. (I guess the Debian package only sets up the vi alternative because vim.tiny might not do 100% of what one would expect from vim.)

#11 Updated by anonym 2017-09-13 13:42:13

• Status changed from In Progress to Fix committed
• Assignee deleted (anonym)
• % Done changed from 80 to 100
• QA Check changed from Ready for QA to Pass

#12 Updated by anonym 2017-09-28 18:49:57

• Status changed from Fix committed to Resolved