xiaoyeli / superlu

Supernodal sparse direct solver. https://portal.nersc.gov/project/sparse/superlu/
Other
275 stars 96 forks source link

hidden files in release tarball #11

Closed nschloe closed 8 years ago

nschloe commented 8 years ago

The SuperLU 5.2.0 tarball contains lots of hidden files which probably shouldn't be there. In ./SRC alone:

$ ls -l SRC/._* | wc -l
195

The GitHub-generated tarball is a bit smaller in size and could probably be swapped in for the one distributed on the website.

xiaoyeli commented 8 years ago

When I did the same list command, it doesn't show any hidden files.

On Thu, May 19, 2016 at 2:59 AM, Nico Schlömer notifications@github.com wrote:

The SuperLU 5.2.0 tarball http://crd-legacy.lbl.gov/%7Exiaoye/SuperLU/superlu_5.2.0.tar.gz contains lots of hidden files which probably shouldn't be there. In ./SRC alone:

$ ls -l SRC/._* | wc -l 195

The GitHub-generated tarball https://github.com/xiaoyeli/superlu/releases/tag/v5.2.0 is about half the size and could probably be swapped in for the one distributed on the website http://crd-legacy.lbl.gov/%7Exiaoye/SuperLU/.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/xiaoyeli/superlu/issues/11

nschloe commented 8 years ago

Strange.

Here's exactly what I did:

$ wget http://crd-legacy.lbl.gov/~xiaoye/SuperLU/superlu_5.2.0.tar.gz
[...]
$ tar xf superlu_5.2.0.tar.gz
$ ls -l SuperLU_5.2.0/SRC/._* | wc -l
195

Specifically, I see

$ cat SuperLU_5.2.0/SRC/._icmax1.c
Mac OS X         2��ATTR��S�Scom.dropbox.attributesx��VJ)�/Hʯ�O��I�L���ON�Q�R�V�ML����%����RK�
                                                                                                            %�����HS���B�2Ê�b��”@[[���Z�~
xiaoyeli commented 8 years ago

Now I see them. I have no idea how they are generated. They are not there in my local version. I will make sure to clean up in the next patch, coming soon.

On Fri, May 20, 2016 at 7:51 AM, Nico Schlömer notifications@github.com wrote:

Strange.

Here's exactly what I did:

$ wget http://crd-legacy.lbl.gov/~xiaoye/SuperLU/superlu_5.2.0.tar.gz [...] $ tar xf superlu_5.2.0.tar.gz $ ls -l SuperLU5.2.0/SRC/.* | wc -l 195

Specifically, I see

$ cat SuperLU_5.2.0/SRC/._icmax1.c ��Mac OS X � 2���ATTR��S��S�com.dropbox.attributesx��VJ)�/Hʯ�O��I�L���ON�Q�R�V�ML�����%�����R�K�� %�����HS���B�2Ê�b���@[[���Z�~

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/xiaoyeli/superlu/issues/11#issuecomment-220627387

nschloe commented 8 years ago

(I know four pull requests which wouldn't mind slipping into the next patch release either. ;))

xiaoyeli commented 8 years ago

I made a new tar ball v5.2.1, including your patches.

On Fri, May 20, 2016 at 8:56 AM, Nico Schlömer notifications@github.com wrote:

(I know four pull requests which wouldn't mind slipping into the next patch release either. ;))

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/xiaoyeli/superlu/issues/11#issuecomment-220645537

nschloe commented 8 years ago

The .dropbox.attr files are still there (just like on git master), but other than that it's looking good.

xiaoyeli commented 8 years ago

Just removed those. I don't know how they came up in the first place.

On Mon, May 23, 2016 at 3:01 AM, Nico Schlömer notifications@github.com wrote:

The .dropbox.attr files are still there (just like on git master), but other than that it's looking good.

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/xiaoyeli/superlu/issues/11#issuecomment-220938375