[aur-general] Wrote Small AUR Helper

Nathan Owens

14 Jan 2011 14 Jan '11

3:57 a.m.

I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

Show replies by date

Brad Fanella

14 Jan 14 Jan

2:46 a.m.

On Thu, Jan 13, 2011 at 8:57 PM, Nathan Owens wrote:

...

I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

Looks promising. Hopefully the C++ will give the helper a speed boost. :-) But, err... https://bbs.archlinux.org/viewforum.php?id=27

Nathan Owens

4:24 a.m.

...

On Thu, Jan 13, 2011 at 8:57 PM, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

Looks promising. Hopefully the C++ will give the helper a speed boost. :-)

But, err... https://bbs.archlinux.org/viewforum.php?id=27 After I get an idea how to do a search function, I would add it to the

On 01/13/2011 07:46 PM, Brad Fanella wrote: program. Though I am of course very new to C++ since it is my first language to try and learn. Thanks for the kind reply to it.

Thomas Dziedzic

3:08 a.m.

On Thu, Jan 13, 2011 at 9:24 PM, Nathan Owens wrote:

...

On 01/13/2011 07:46 PM, Brad Fanella wrote:

...
On Thu, Jan 13, 2011 at 8:57 PM, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

Looks promising. Hopefully the C++ will give the helper a speed boost. :-)

But, err... https://bbs.archlinux.org/viewforum.php?id=27

After I get an idea how to do a search function, I would add it to the program. Though I am of course very new to C++ since it is my first language to try and learn.

Thanks for the kind reply to it.

Hey, looks like a great start to c++ :) One thing to look into though is to see if you can interface with the aur rpc interface. This would be a nice "next step" towards a proper aur helper imo.

Nathan Owens

5:01 a.m.

On 01/13/2011 08:08 PM, Thomas Dziedzic wrote:

...

On Thu, Jan 13, 2011 at 9:24 PM, Nathan Owens wrote:

...
...
On Thu, Jan 13, 2011 at 8:57 PM, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

Looks promising. Hopefully the C++ will give the helper a speed boost. :-)

But, err... https://bbs.archlinux.org/viewforum.php?id=27 After I get an idea how to do a search function, I would add it to the

On 01/13/2011 07:46 PM, Brad Fanella wrote: program. Though I am of course very new to C++ since it is my first language to try and learn.

Thanks for the kind reply to it.

Hey, looks like a great start to c++ :) One thing to look into though is to see if you can interface with the aur rpc interface. This would be a nice "next step" towards a proper aur helper imo. Yep :) I am currently trying to figure out what I can use to be able to do that with. Though that would be I guess my next release.

Jonathan Conder

3:41 a.m.

On 14 January 2011 15:57, Nathan Owens wrote:

...

I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

So far so good, just a few pointers: - The struct Aur isn't necessary - the file field is never used. You should remove Aur:: from the getFile definition to make it an ordinary function. - The return type of getFile should be void, not char. I would suggest passing the -Wall and -pedantic options to g++ so the compiler would catch these types of issues. - The only headers I think you need are <iostream>, and <string> (not , which is a C header). I might have missed something though. - As a next step I would use a library to download the file, rather than calling wget which isn't very efficient. You might choose libcurl, libfetch, libsoup, Qt, or maybe something else. - Although it isn't needed yet, I would suggest adding a break statement after case 1, to avoid errors when you add more cases. Hope this helps, Jonathan

Nathan Owens

5:13 a.m.

On 01/13/2011 08:41 PM, Jonathan Conder wrote:

...

On 14 January 2011 15:57, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

So far so good, just a few pointers:

- The struct Aur isn't necessary - the file field is never used. You should remove Aur:: from the getFile definition to make it an ordinary function. - The return type of getFile should be void, not char. I would suggest passing the -Wall and -pedantic options to g++ so the compiler would catch these types of issues. - The only headers I think you need are<iostream>, and<string> (not, which is a C header). I might have missed something though. - As a next step I would use a library to download the file, rather than calling wget which isn't very efficient. You might choose libcurl, libfetch, libsoup, Qt, or maybe something else. - Although it isn't needed yet, I would suggest adding a break statement after case 1, to avoid errors when you add more cases.

Hope this helps, Jonathan Refresh to code. I updated after I noticed the AUR:: stuff wasnt needed. So I changed the struct to class

Nathan Owens

7:36 a.m.

On 01/13/2011 10:13 PM, Nathan Owens wrote:

...

On 01/13/2011 08:41 PM, Jonathan Conder wrote:

...
On 14 January 2011 15:57, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

So far so good, just a few pointers:

- The struct Aur isn't necessary - the file field is never used. You should remove Aur:: from the getFile definition to make it an ordinary function. - The return type of getFile should be void, not char. I would suggest passing the -Wall and -pedantic options to g++ so the compiler would catch these types of issues. - The only headers I think you need are<iostream>, and<string> (not, which is a C header). I might have missed something though. - As a next step I would use a library to download the file, rather than calling wget which isn't very efficient. You might choose libcurl, libfetch, libsoup, Qt, or maybe something else. - Although it isn't needed yet, I would suggest adding a break statement after case 1, to avoid errors when you add more cases.

Hope this helps, Jonathan Refresh to code. I updated after I noticed the AUR:: stuff wasnt needed. So I changed the struct to class Also just remembered, there was a discussion about having a AUR helper in community, but since others actually installed it for the user they wouldn't be added to community. What I thought about, maybe after it gets improved and enough votes, I wonder if it might be able to be added to community.

Thomas Dziedzic

7:12 a.m.

On Fri, Jan 14, 2011 at 12:36 AM, Nathan Owens wrote:

...

On 01/13/2011 10:13 PM, Nathan Owens wrote:

...
On 01/13/2011 08:41 PM, Jonathan Conder wrote:

...
On 14 January 2011 15:57, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

So far so good, just a few pointers:

- The struct Aur isn't necessary - the file field is never used. You should remove Aur:: from the getFile definition to make it an ordinary function. - The return type of getFile should be void, not char. I would suggest passing the -Wall and -pedantic options to g++ so the compiler would catch these types of issues. - The only headers I think you need are<iostream>, and<string> (not, which is a C header). I might have missed something though. - As a next step I would use a library to download the file, rather than calling wget which isn't very efficient. You might choose libcurl, libfetch, libsoup, Qt, or maybe something else. - Although it isn't needed yet, I would suggest adding a break statement after case 1, to avoid errors when you add more cases.

Hope this helps, Jonathan

Refresh to code. I updated after I noticed the AUR:: stuff wasnt needed. So I changed the struct to class

Also just remembered, there was a discussion about having a AUR helper in community, but since others actually installed it for the user they wouldn't be added to community. What I thought about, maybe after it gets improved and enough votes, I wonder if it might be able to be added to community.

Sorry but aur helpers aren't allowed in [community], and that really is a hard rule.

Nathan Owens

8:44 a.m.

On 01/14/2011 12:12 AM, Thomas Dziedzic wrote:

...

On Fri, Jan 14, 2011 at 12:36 AM, Nathan Owens wrote:

...
On 01/13/2011 10:13 PM, Nathan Owens wrote:

...
On 01/13/2011 08:41 PM, Jonathan Conder wrote:

...
On 14 January 2011 15:57, Nathan Owens wrote:

...
I wrote a little AUR helper in C++, though currently it only downloads the tarball if the package name is valid. I am new to C++ and wanted to know if you thought the code was good. I was thinking about putting it in AUR, but figured I would asked for opinion first. Here is the url for the code: http://pastebin.com/aSW1awD4

So far so good, just a few pointers:

- The struct Aur isn't necessary - the file field is never used. You should remove Aur:: from the getFile definition to make it an ordinary function. - The return type of getFile should be void, not char. I would suggest passing the -Wall and -pedantic options to g++ so the compiler would catch these types of issues. - The only headers I think you need are<iostream>, and<string> (not, which is a C header). I might have missed something though. - As a next step I would use a library to download the file, rather than calling wget which isn't very efficient. You might choose libcurl, libfetch, libsoup, Qt, or maybe something else. - Although it isn't needed yet, I would suggest adding a break statement after case 1, to avoid errors when you add more cases.

Hope this helps, Jonathan Refresh to code. I updated after I noticed the AUR:: stuff wasnt needed. So I changed the struct to class Also just remembered, there was a discussion about having a AUR helper in community, but since others actually installed it for the user they wouldn't be added to community. What I thought about, maybe after it gets improved and enough votes, I wonder if it might be able to be added to community.

Sorry but aur helpers aren't allowed in [community], and that really is a hard rule. That is fine, just thought I remember something about the reason why was because most of the others install the packages.

4854

Age (days ago)

4854

Last active (days ago)

List overview

Download

9 comments

4 participants

participants (4)

Brad Fanella
Jonathan Conder
Nathan Owens
Thomas Dziedzic