Request 1159306 (superseded)

This request is superseded by request 1159777 (Show diff)

Overview

Request 1159306 superseded

add fswatch to openSUSE Factory

Will be a file watcher backend for neovim 0.10.0
https://github.com/neovim/neovim/pull/27347

Created by mvyskocil 2 months ago
In state superseded
Superseded by 1159777
Open review for opensuse-review-team
Open review for openSUSE:Factory:Staging:adi:38

Submit fswatch

Comments for request 1159306 2

Dominique Leuenberger (dimstar) - 2 months ago

why -locale ? All packages in the distro use %lang_package which in turn creates a -lang package.

I don't see a reason to deviate from that naming convention here (as there is no main package, the lib%{name}13 mus provide %{name} to make the generated -lang package installable

Michal Vyskocil (mvyskocil) - 2 months ago

Thanks for a review. Did not know about %lang_package. Will post an another submit request.

as there is no main package

I believe this is misunderstanding. There is a /usr/bin/fswatch installed byfswatch package. The installation of built packages worked well.

sudo rpm -ivh rpms/fswatch-lang-1.17.1-0.noarch.rpm rpms/fswatch-1.17.1-0.x86_64.rpm rpms/libfswatch13-1.17.1-0.x86_64.rpm
Verifying...                          ################################# [100%]
Preparing...                          ################################# [100%]
Updating / installing...
   1:libfswatch13-1.17.1-0            ################################# [ 33%]
   2:fswatch-1.17.1-0                 ################################# [ 67%]
   3:fswatch-lang-1.17.1-0            ################################# [100%]

Dominique Leuenberger (dimstar) - 2 months ago

ah - indeed; that was a mis-understanging; and I thought you'd avoided the %lang_package macro as it creates a dep on %{name} = %{version}; sorry for that