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
cliquer: init at 1.21 #38795
cliquer: init at 1.21 #38795
Conversation
homepage = https://users.aalto.fi/~pat/cliquer.html; | ||
description = "Routines for clique searching"; | ||
longDescription = '' | ||
Cliquer is a set of C routines for finding cliques in an arbitrary weighted graph. It uses an exact branch-and-bound algorithm developed by Patric Östergård. It is designed with the aim of being efficient while still being flexible and easy to use. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line breaks
''; | ||
license = with licenses; [ gpl2 ]; | ||
maintainers = with maintainers; [ timokau ]; | ||
platforms = platforms.all; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested?
version = "1.21"; | ||
name = "cliquer-${version}"; | ||
|
||
# autotoolized version of the original cliquer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this package should have the name cliquer then. And the homepage should also be changed accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its still the same package with the same functionality, just an improved / updated build. I think thats an implementation detail that would only confuse users that want to install cliquer.
However if you prefer a name change and have a different name in mind, I'm not strongly opposed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering that Arch is also using autocliquer, this seems alright after all: https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/cliquer#n15
However please add a downloadPage
pointing to autocliquer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could use
downloadPage = src.meta.homepage;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
It is designed with the aim of being efficient while still being flexible and | ||
easy to use. | ||
''; | ||
license = with licenses; [ gpl2 ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please get rid of the list and just use licenses.gpl2Plus
:
Cliquer is licensed under the GNU General Public License as published
by the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
(from the README in the tarball)
@GrahamcOfBorg build cliquer |
Success on x86_64-linux (full log) Attempted: cliquer Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: cliquer Partial log (click to expand)
|
Motivation for this change
Package cliquer.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)