[opensuse-factory] mdds package in devel:libraries:c_c++
Hi there,
I would like to request an addition of mdds library to the
devel:libraries:c_c++ project. Mdds stands for Multi-Dimensional Data
Structure, and is a collection of C++ templates that provide optimized
storage and lookup of multi-dimensional data. I am the author and
maintainer of this library, and I am willing to maintain this package
for SUSE.
I have already packaged this library, which is now sitting in my
personal home project. I would like that moved into a more generic
project, such as devel:libraries:c_c++.
Here is where it is packaged right now:
https://build.opensuse.org/package/show?package=mdds&project=home%
3Akohei_yoshida
The project home page is here:
http://code.google.com/p/multidimalgorithm/
As this is my first attempt to have my package included in the SUSE
distribution, any advice or hand-holding of how to get it in would be
much appreciated.
Best regards,
Kohei
--
Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc.
Hi, On 05/28/2010 02:50 PM, Kohei Yoshida wrote:
Hi there,
I would like to request an addition of mdds library to the devel:libraries:c_c++ project. Mdds stands for Multi-Dimensional Data Structure, and is a collection of C++ templates that provide optimized storage and lookup of multi-dimensional data. I am the author and maintainer of this library, and I am willing to maintain this package for SUSE.
I have already packaged this library, which is now sitting in my personal home project. I would like that moved into a more generic project, such as devel:libraries:c_c++.
Here is where it is packaged right now:
https://build.opensuse.org/package/show?package=mdds&project=home% 3Akohei_yoshida
The project home page is here:
http://code.google.com/p/multidimalgorithm/
As this is my first attempt to have my package included in the SUSE distribution, any advice or hand-holding of how to get it in would be much appreciated.
You've already taken the first couple of steps, built the package and sent a message to the list. You also need a submit request. Checkout the following blog: http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu... Robert -- Robert Schweikert MAY THE SOURCE BE WITH YOU Software Engineer Consultant LINUX rschweikert@novell.com 781-464-8147 Novell Making IT Work As One -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-factory+help@opensuse.org
On Fri, 2010-05-28 at 15:04 -0400, Robert Schweikert wrote:
You've already taken the first couple of steps, built the package and sent a message to the list. You also need a submit request. Checkout the following blog:
http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu...
Thanks. I've just submitted a request to move mdds into devel:libraries:c_c++. Let's see what happens there. Kohei -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-factory+help@opensuse.org
On Fri, May 28, 2010 at 03:37:33PM -0400, Kohei Yoshida wrote:
On Fri, 2010-05-28 at 15:04 -0400, Robert Schweikert wrote:
You've already taken the first couple of steps, built the package and sent a message to the list. You also need a submit request. Checkout the following blog:
http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu...
Thanks. I've just submitted a request to move mdds into devel:libraries:c_c++. Let's see what happens there.
You could check the buildlog and try to fix some of the warnings at the end. e.g. - you copy files to $RPM_BUILD_ROOT in the %build section This should be moved to the %install section. - You seem to include test cases... Perhaps build and run them in a "%check" section. Ciao, Marcus -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-factory+help@opensuse.org
Hello Marcus, On Fri, 2010-05-28 at 21:40 +0200, Marcus Meissner wrote:
You could check the buildlog and try to fix some of the warnings at the end.
I just worked on it to remove all RPMLINT warnings.
- you copy files to $RPM_BUILD_ROOT in the %build section
This should be moved to the %install section.
Done!
- You seem to include test cases... Perhaps build and run them in a "%check" section.
This is a good idea; however, I seem to be having some problem with
this, as the build has hard time finding g++.
If this is not a requirement, perhaps I could work on this as an
improvement for the next version ? This package is not fully
autoconf'ed yet (i.e. no 'make test'), so, implementing check would be a
bit of a challenge.
What do you think?
Kohei
--
Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc.
On Fri, 2010-05-28 at 15:37 -0400, Kohei Yoshida wrote:
On Fri, 2010-05-28 at 15:04 -0400, Robert Schweikert wrote:
You've already taken the first couple of steps, built the package and sent a message to the list. You also need a submit request. Checkout the following blog:
http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu...
Thanks. I've just submitted a request to move mdds into devel:libraries:c_c++. Let's see what happens there.
Just to clarify (since my request has just been declined), this is a
header-only library since all of what it provides are C++ templates.
So, there is no compilation involved.
--
Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc.
* Kohei Yoshida (kyoshida@novell.com) [20100529 02:15]:
Just to clarify (since my request has just been declined), this is a header-only library since all of what it provides are C++ templates. So, there is no compilation involved.
Yes, he didn't check the package. I've made a few changes and additions and will do a sr as soon as building/installing succeeds. Please tell me what you think of them. Philipp -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-factory+help@opensuse.org
On Mon, 2010-05-31 at 13:53 +0200, Philipp Thomas wrote:
* Kohei Yoshida (kyoshida@novell.com) [20100529 02:15]:
Just to clarify (since my request has just been declined), this is a header-only library since all of what it provides are C++ templates. So, there is no compilation involved.
Yes, he didn't check the package. I've made a few changes and additions and will do a sr as soon as building/installing succeeds. Please tell me what you think of them.
They look good to me. Thanks a lot. :-)
The only thing I would change is to remove
test.recset.perf
test.st.perf
test.stl
since they are only designed to run during development for performance
tuning, and are not really designed to give you "pass" or "fail" during
installation.
Having said that, (since I'm still a newbie in packaging) if you think
those tests still make sense during installation, I'd be fine with it.
Kohei
--
Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc.
participants (4)
-
Kohei Yoshida -
Marcus Meissner -
Philipp Thomas -
Robert Schweikert