[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#859238: RFS: microsoft-gsl/0.1~2017.03.20~git16a6a41-1



04.04.2017 03:22, Mattia Rizzolo пишет:
As others said already, 'microsoft' in the package name is a sad
situation.  Personally, is just a can of worms I do not want to open for
so little, so please rename it to something else (I like 'ms-gsl').

I changed package name to ms-gsl as you want (libmsgsl-dev for .deb package). These names sound as good as the previous ones, I think. Anyway, I don't know much about these law things.

 Version         : 0.1~2017.03.20~git16a6a41-1

I recommend using 0 instead of 0.1 as base version.

Okay, I see this is a good idea. I also updated to a new version as well.

As I said privately, I'd enjoy having a git repository for this :)
Here I feel you could enjoy even more baing the repo out of upstream
git (see an example in the dehydrated package); or you can see my
pencil2d package for an example of a thing building tarball out of
upstream git, ready to be committed; as you prefer.

I temporally uploaded my repo to GitHub [1]. I believe alioth.debian.org will be a better location. I've registered there (my username is mymedia-guest) just now.

The first commit in that repository corresponds to what I uploaded to mentors.debian.net. I made final modifications (for this stage) in the last, 8dec145.

I also made get-orig-source target in d/rules. It clones the upstream repository and pack it in a tar archive. I made it because I hadn't found any tarball on upstream GitHub.

* test building, I noticed it didn't take advantage of my quad-core
  system; why didn't you use compat level 10?

I'm Ubuntu user, but there is only debhelper of version 9 in ubuntu repositories. So I added --parallel option into d/rules file as an interim solution.

* please send that UnitTest.patch upstream; that's clearly one of those
  cases their stupid system with a case-insensitive file system tricked
  them…
* that empty directory tests/unittest-cpp, why didn't you remove it?

It seems the upstream doesn't need this patch because they use a last version of UnitTeset++ framework where the header has capital letters. Was I supposed to delete that directory? I thought it was in the upstream repo and I shouldn't have touch it.

04.04.2017 20:25, PICCORO McKAY Lenz пишет:
this "library" does not provide real funtionallity, its only to code "as
moscosoft like"

You got the wrong idea. This library is an implementation of C++ Core Guidelines written by Bjarne Stroustrup and Herb Sutter. It could very well be included into a future C++ Standard. You have not to regard this as a creature of evil Microsoft. See an old announce [2].

Theoretically, this may not be the only implementation, but I could not find another one. If you get a replacement, I'll be very glad. But right now your attacks are unconstructive and blatantly false.

[1] https://github.com/mymedia2/ms-gsl
[2] https://isocpp.org/blog/2015/09/bjarne-stroustrup-announces-cpp-core-guidelines


Reply to: