Re: [pacman-dev] [PATCH] test version range restrictions corner case

2016-02-19 Thread Dominik Fischer
Bug filed at https://bugs.archlinux.org/task/48252 . Thanks for your consideration.

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2016-01-17 Thread Allan McRae
On 18/01/16 02:37, Dominik Fischer wrote: > Thanks for the pull. I saw it appeares as XFAIL in the master branch. Is > this > enough or should I also file this in the bug tracker [1], so that the issue > will not be forgotten? > > [1]: https://bugs.archlinux.org/index.php?project=3 > Feel free t

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2016-01-17 Thread Dominik Fischer
Thanks for the pull. I saw it appeares as XFAIL in the master branch. Is this enough or should I also file this in the bug tracker [1], so that the issue will not be forgotten? [1]: https://bugs.archlinux.org/index.php?project=3

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2016-01-03 Thread Allan McRae
On 08/12/15 06:37, Dominik Fischer wrote: > The test introduced herein illustrates a behavior that may be unexpected > to package writers. > > It creates a package "pkg3" that is configured to depend on a > "dependency" which version is between 3 and 4, inclusive. Two other > packages are already

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2015-12-08 Thread Dominik Fischer
If this is the intended way to specify a version range, I don't see any need to introduce a dedicated syntax. It just has to be ensured that when checking dependencies, all requirements for a package have to be satisfied by the same package. Then, the given scenario would be rejected because "pkg1

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2015-12-08 Thread Emil Lundberg
> `(3.14,4)` for >3.14 and <=4. Whoops! That would be <4, not <=4.

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2015-12-08 Thread Emil Lundberg
> Nonetheless, what a package writer intends when specifying > > depends=('dependency>=3' 'dependency<=4') > > is most probably that pacman should only allow this package to be > installed when there indeed is a package present that provides a version > of "dependency" that lies _between_ 3 and 5

Re: [pacman-dev] [PATCH] test version range restrictions corner case

2015-12-08 Thread Florian Pritz
On Mon, 7 Dec 2015 21:37:09 +0100 Dominik Fischer wrote: > This behavior may be intended though. If so, this corner case should > perhaps be noted in the documentation. Please review and comment. Actually that is exactly what some of our packages want to do so I'd consider it a bug. Nice find!

[pacman-dev] [PATCH] test version range restrictions corner case

2015-12-07 Thread Dominik Fischer
The test introduced herein illustrates a behavior that may be unexpected to package writers. It creates a package "pkg3" that is configured to depend on a "dependency" which version is between 3 and 4, inclusive. Two other packages are already present, providing "dependency" in version 2 and 5, r