Re: [dev] [PATCH] Use git to create dist archives.

2015-03-16 Thread Alex Pilon
 On Mon, 16 Mar 2015 22:18:33 +0100 Alex Pilon a...@alexpilon.ca wrote:
  Less code, and people notice when things aren't in the repo unlike a
  little used target.

On Mon, Mar 16, 2015 at 10:18:33PM +0100, Christoph Lohmann wrote:
 That's too simple-minded. You are adding a dependency on git.

I considered that, but thought it the lesser evil. *Ideally*, build
systems and version control systems shouldn't be tightly coupled, if at
all. I have my own prejudices, and we can agree to cordially disagree on
such a trivial and here insignificant matter.


pgpWc9xDbS8yY.pgp
Description: PGP signature


Re: [dev] [dwm] Add i3 to differences in http://dwm.suckless.org/

2015-03-16 Thread Anselm R Garbe
On 16 March 2015 at 05:58, Ivan Tham ivanthamjun...@gmail.com wrote:
 I want to learn C, so now I am waiting for c.learncodethehardway.org
 to release as other book may seem a bit too long for me.

You should rather learn C from its inventors:

http://en.wikipedia.org/wiki/The_C_Programming_Language

BR,
Anselm



Re: [dev] [st] Problems when typing

2015-03-16 Thread Amadeus Folego
On Mon, Mar 16, 2015 at 02:12:16PM +0800, Ivan Tham wrote:
 Sorry for that, but I didn't say that I am unwilling to read a C book.

Yes, you did, you are inventing reasons to justify your actions:

 I want to learn C, so now I am waiting for c.learncodethehardway.org
 to release as other book may seem a bit too long for me.

 Thanks, but it takes time reading it.

 (this will spoil my eyesight)

Actually you don't need to read a book or a manual to post questions,
you just need to be reasonable and not annoying.

You should respect people and not waste their time and efforts asking
every question imaginable without doing the minimal effort to learn.

Chris is exaggerating for some reason: being funny, extreme or
something like that. But he is right in the sense that we should not
accept being treated like someone's personal google.



Re: [dev] [dwm] Add i3 to differences in http://dwm.suckless.org/

2015-03-16 Thread Ivan Tham
Thanks. I will.

On 3/16/15, Anselm R Garbe garb...@gmail.com wrote:
 On 16 March 2015 at 05:58, Ivan Tham ivanthamjun...@gmail.com wrote:
 I want to learn C, so now I am waiting for c.learncodethehardway.org
 to release as other book may seem a bit too long for me.

 You should rather learn C from its inventors:

 http://en.wikipedia.org/wiki/The_C_Programming_Language

 BR,
 Anselm





[dev] [PATCH] Handle pasting of empty selection.

2015-03-16 Thread Alex Pilon
Otherwise, pasting the X11 primary selection when empty results an
error and Xlib forcibly exits.
---
 st.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/st.c b/st.c
index ac91d9f..cd470e7 100644
--- a/st.c
+++ b/st.c
@@ -1012,6 +1012,8 @@ selnotify(XEvent *e) {
 
ofs = 0;
xsev = (XSelectionEvent *)e;
+   if (xsev-property == None)
+   return;
do {
if(XGetWindowProperty(xw.dpy, xw.win, xsev-property, ofs,
BUFSIZ/4, False, AnyPropertyType,
-- 
2.3.3



pgpRKRgWMu03_.pgp
Description: PGP signature


[dev] [sbase] No -lrt in config.mk

2015-03-16 Thread Eric Pruitt
Is there a reason -lrt is missing from config.mk in sbase? I recall
seeing some messages saying that OpenBSD lacks -lrt support (or
something to that effect), but since its a POSIX library and is required
to compile find, it seems like it should be present by default.
Searching the list for -lrt, I see Roberto submitted a patch on
February 20th; why wasn't it applied?

Eric



[dev] [PATCH] Use git to create dist archives.

2015-03-16 Thread Alex Pilon
It's easy to forget things, like fixed with the following out of tree
patch.

commit ec6937b989a171e872c58bc60f8b21bd25880cd5
Author: M Farkas-Dyck strake...@gmail.com
Date:   Wed Dec 10 14:29:34 2014 -0500

dist arg.h; needed to build

diff --git a/Makefile b/Makefile
index 52af636..25719f1 100644
--- a/Makefile
+++ b/Makefile
@@ -34,7 +34,7 @@ clean:
 dist: clean
@echo creating dist tarball
@mkdir -p st-${VERSION}
-   @cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 
${SRC} st-${VERSION}
+   @cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 
${SRC} arg.h st-${VERSION}
@tar -cf st-${VERSION}.tar st-${VERSION}
@gzip st-${VERSION}.tar
@rm -rf st-${VERSION}

Less code, and people notice when things aren't in the repo unlike a
little used target.
---
 Makefile | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/Makefile b/Makefile
index 4d6a8a9..01e50ef 100644
--- a/Makefile
+++ b/Makefile
@@ -33,11 +33,7 @@ clean:
 
 dist: clean
@echo creating dist tarball
-   @mkdir -p st-${VERSION}
-   @cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 
${SRC} st-${VERSION}
-   @tar -cf st-${VERSION}.tar st-${VERSION}
-   @gzip st-${VERSION}.tar
-   @rm -rf st-${VERSION}
+   @git archive --prefix=st-${VERSION}/ HEAD | gzip -c  
st-${VERSION}.tar.gz
 
 install: all
@echo installing executable file to ${DESTDIR}${PREFIX}/bin
-- 
2.3.3




Re: [dev] [PATCH] [sbase] No blkcnt_t on OpenBSD

2015-03-16 Thread Hiltjo Posthuma
On Mon, Mar 16, 2015 at 8:47 PM, Steven Dee st...@smartercode.net wrote:
 ---
  du.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/du.c b/du.c
 index 6ecebc9..962e61c 100644
 --- a/du.c
 +++ b/du.c
 @@ -29,7 +29,7 @@ printpath(size_t n, const char *path)
  }

  static size_t
 -nblks(blkcnt_t blocks)
 +nblks(size_t blocks)
  {
 return (512 * blocks + blksize - 1) / blksize;
  }
 --
 1.9.3


This should also be signed: blkcnt_t and off_t shall be signed
integer types.[0]

On my OpenBSD 5.6 install st_blocks is also defined as int64_t, but on
-current this is fixed like Dimitris said.

Kind regards,
Hiltjo

[0] http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/sys_types.h.html



Re: [dev] [PATCH] Use git to create dist archives.

2015-03-16 Thread Christoph Lohmann
Greetings.

On Mon, 16 Mar 2015 22:18:33 +0100 Alex Pilon a...@alexpilon.ca wrote:
 commit ec6937b989a171e872c58bc60f8b21bd25880cd5
 Author: M Farkas-Dyck strake...@gmail.com
 Date:   Wed Dec 10 14:29:34 2014 -0500
 
 dist arg.h; needed to build
 
 diff --git a/Makefile b/Makefile
 index 52af636..25719f1 100644
 --- a/Makefile
 +++ b/Makefile
 @@ -34,7 +34,7 @@ clean:
  dist: clean
 @echo creating dist tarball
 @mkdir -p st-${VERSION}
 - @cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 
 ${SRC} st-${VERSION}
 + @cp -R LICENSE Makefile README config.mk config.def.h st.info st.1 
 ${SRC} arg.h st-${VERSION}
 @tar -cf st-${VERSION}.tar st-${VERSION}
 @gzip st-${VERSION}.tar
 @rm -rf st-${VERSION}

Applied.

 Less code, and people notice when things aren't in the repo unlike a
 little used target.

That's too simple-minded. You are adding a dependency on git.


Sincerely,

Christoph Lohmann




Re: [dev] [st] Problems when typing

2015-03-16 Thread Ivan Tham
Sorry for that, but I didn't say that I am unwilling to read a C book.

On 3/16/15, Alexander S. alex0pla...@gmail.com wrote:
 2015-03-16 8:36 GMT+03:00 Christoph Lohmann 2...@r-36.net:
 Sincerely,

 Christoph Lohmann
 Wow, didn't ever seen ye that full of yerself, 20h. Making up
 challenges for random people at the same time you tell them to never
 come back, eh? Must make a ton of sense to you.





Re: [dev] [faq] How do I push to st repository?

2015-03-16 Thread Greg Reagle
On Sun, Mar 15, 2015, at 11:02 AM, Markus Teich wrote:
 read http://git-scm.com/book/en/v2 at least until chapter 3.

Thanks for this recommendation.  I will read it.  (I know it was
intended for someone else.)  I have a pretty good basic understanding of
git and I've created, merged, and deleted branches, for instance, but a
good solid education in git will be very useful.  Thanks again.

-- 
http://www.fastmail.com - Does exactly what it says on the tin




[dev] [PATCH] [sbase] No blkcnt_t on OpenBSD

2015-03-16 Thread Steven Dee
---
 du.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/du.c b/du.c
index 6ecebc9..962e61c 100644
--- a/du.c
+++ b/du.c
@@ -29,7 +29,7 @@ printpath(size_t n, const char *path)
 }
 
 static size_t
-nblks(blkcnt_t blocks)
+nblks(size_t blocks)
 {
return (512 * blocks + blksize - 1) / blksize;
 }
-- 
1.9.3




Re: [dev] [PATCH] Handle pasting of empty selection.

2015-03-16 Thread Christoph Lohmann
Greetings.

On Mon, 16 Mar 2015 20:31:48 +0100 Alex Pilon a...@alexpilon.ca wrote:
 Otherwise, pasting the X11 primary selection when empty results an
 error and Xlib forcibly exits.

Thanks for sending the patch. I was debuggint that crash myself already.
You were faster. It has been applied.


Sincerely,

Christoph Lohmann




Re: [dev] [PATCH] [sbase] No blkcnt_t on OpenBSD

2015-03-16 Thread Dimitris Papastamos
On Mon, Mar 16, 2015 at 03:47:21PM -0400, Steven Dee wrote:
 ---
  du.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/du.c b/du.c
 index 6ecebc9..962e61c 100644
 --- a/du.c
 +++ b/du.c
 @@ -29,7 +29,7 @@ printpath(size_t n, const char *path)
  }
  
  static size_t
 -nblks(blkcnt_t blocks)
 +nblks(size_t blocks)
  {
   return (512 * blocks + blksize - 1) / blksize;
  }

Has been fixed on -current.



Re: [dev] [st] Problems when typing

2015-03-16 Thread FRIGN
On Mon, 16 Mar 2015 08:54:39 +0300
Alexander S. alex0pla...@gmail.com wrote:

Hey Alexander,

 Wow, didn't ever seen ye that full of yerself, 20h. Making up
 challenges for random people at the same time you tell them to never
 come back, eh? Must make a ton of sense to you.

Christoph is totally right. These people are wasting our time and it
should be known by now that suckless is not a customer satisfaction
hotline.
He also did not tell him to never come back. He made it clear to him
what raw standards there are to be remotely able to actually solve
a problem using common sense and a programming language like C.
These bug-reports are pissing me off as well, but I don't respond
to them. Don't attack Christoph for doing the dirty work here.

I hate to bring this up, but this is yet another reason to have
separate mailing lists for development and bug-reports. Since
we've somehow become cool in the Arch-community, the number of
spam-threads on this list has grown beyond control.

Cheers

FRIGN

-- 
FRIGN d...@frign.de