You are not logged in.

#1 2008-09-20 01:07:43

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

fbrun in AUR

I use openbox and don't like gmrun, so i make a package for fbrun from Fluxbox.

Here: http://aur.archlinux.org/packages.php?ID=20075

Options:

-font [font name]           Text font
   -title [title name]         Set title
   -text [text]                Text input
   -w [width]                  Window width in pixels
   -h [height]                 Window height in pixels
   -display [display string]   Display name
   -pos [x] [y]                Window position in pixels
   -nearmouse                  Window position near mouse
   -fg [color name]            Foreground text color
   -bg [color name]            Background color
   -na                         Disable antialias
   -hf [history file]          History file to load (default ~/.fbrun_history)
   -help                       Show this help

First release: patch to change history file from ~/.fluxbox/fbrun_history to ~/.fbrun_history

Screen: 20080919220712282x46scruy6.th.png

In screenshot i use the options "fbrun -tltle Executar -font Sans:9 -h 23 -w 280


UPDATE 1.5-3: All patches are optional.

Last edited by kramerxiita (2008-09-24 16:11:34)

Offline

#2 2008-09-24 13:47:44

Allan
Pacman
From: Brisbane, AU
Registered: 2007-06-09
Posts: 11,398
Website

Re: fbrun in AUR

kramerxiita wrote:

First release: patch to change history file from ~/.fluxbox/fbrun_history to ~/.fbrun_history

Doesn't it do the opposite?

Offline

#3 2008-09-24 13:57:53

drag0nl0rd
Package Maintainer (PM)
From: Czech Republic
Registered: 2007-10-24
Posts: 127
Website

Re: fbrun in AUR

Ups, accidentaly deleted:

drag0nl0rd wrote:

I like this one, but why this patching? What if someone use it with another WM (=> config file in ~/.fluxbox/)? I think such patch isn't the philosophy of Arch, you may consider to drop it smile
Good work, anyway.

Eh, now I understand. 'fbrun' conflicts with 'fluxbox' and I also thought you move it to ~/.fluxbox... What's wrong with my reading skills today hmm So in this case, the patch isn't that illogical, but anyways, Arch=only as patching as necessary tongue

Allan wrote:

Doesn't it do the opposite?

I think this confused me ... so maybe I still can read smile

Offline

#4 2008-09-24 14:03:35

moljac024
Member
From: Serbia
Registered: 2008-01-29
Posts: 2,676

Re: fbrun in AUR

Honestly, I don't see any advantages over gmrun.


The day Microsoft makes a product that doesn't suck, is the day they make a vacuum cleaner.
--------------------------------------------------------------------------------------------------------------
But if they tell you that I've lost my mind, maybe it's not gone just a little hard to find...

Offline

#5 2008-09-24 16:03:56

Mr Green
Forum Fellow
From: U.K.
Registered: 2003-12-21
Posts: 5,898
Website

Re: fbrun in AUR

yay! gmrun does what it says on the tin!


Mr Green

Offline

#6 2008-09-24 16:05:03

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

Re: fbrun in AUR

moljac024 wrote:

Honestly, I don't see any advantages over gmrun.

And, for you, what the Gmrun advantage over Fbrun?

Offline

#7 2008-09-24 16:18:41

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

Re: fbrun in AUR

Allan wrote:
kramerxiita wrote:

First release: patch to change history file from ~/.fluxbox/fbrun_history to ~/.fbrun_history

Doesn't it do the opposite?

Default: ~/.fluxbox/.fbrun_history

With patch: ~/.fbrun_history.

But is bettler make this patch optional too. I make this later.

But if anyone use Fluxbox, don't need install fbrun. So, why have .fluxbox directory?

Last edited by kramerxiita (2008-09-24 16:19:18)

Offline

#8 2008-09-24 17:51:47

moljac024
Member
From: Serbia
Registered: 2008-01-29
Posts: 2,676

Re: fbrun in AUR

kramerxiita wrote:
moljac024 wrote:

Honestly, I don't see any advantages over gmrun.

And, for you, what the Gmrun advantage over Fbrun?

Tab completion displays the possible results when there are more than one, and it also looks better to me.
To each his own smile


The day Microsoft makes a product that doesn't suck, is the day they make a vacuum cleaner.
--------------------------------------------------------------------------------------------------------------
But if they tell you that I've lost my mind, maybe it's not gone just a little hard to find...

Offline

#9 2008-09-24 20:42:55

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

Re: fbrun in AUR

"Tab completion displays the possible results"

In fbrun too!

But i prefer for window decoration!

Offline

#10 2008-09-24 23:48:24

drag0nl0rd
Package Maintainer (PM)
From: Czech Republic
Registered: 2007-10-24
Posts: 127
Website

Re: fbrun in AUR

kramerxiita wrote:
Allan wrote:

Doesn't it do the opposite?

Default: ~/.fluxbox/.fbrun_history
With patch: ~/.fbrun_history.

Didn't you switch files while making this patch? According to what I see ~/.fbrun_history becomes ~/.fluxbox/fbrun_history ... which is Allan pointing at.

--- main.cc    2008-09-19 20:56:40.000000000 -0300
+++ fluxbox-1.1.1/util/fbrun/main.cc    2008-01-15 07:50:53.000000000 -0200
[snip]
-        "   -hf [history file]          History file to load (default ~/.fbrun_history)"<<endl<<
+        "   -hf [history file]          History file to load (default ~/.fluxbox/fbrun_history)"<<endl<<
         "   -help                       Show this help"<<endl<<endl<<
         "Example: fbrun -fg black -bg white -text xterm -title \"run xterm\""<<endl;
 }
@@ -81,7 +81,7 @@
     string foreground("black");   // text color
     string background("white");   // text background color
     string display_name; // name of the display connection
-    string history_file("~/.fbrun_history"); // command history file
+    string history_file("~/.fluxbox/fbrun_history"); // command history file
kramerxiita wrote:

But is bettler make this patch optional too. I make this later.
But if anyone use Fluxbox, don't need install fbrun. So, why have .fluxbox directory?

Optional is good, although in this case it isn't illogical to drop the .fluxbox directory, as I said.

Offline

#11 2008-09-25 11:34:55

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

Re: fbrun in AUR

drag0nl0rd wrote:
kramerxiita wrote:
Allan wrote:

Doesn't it do the opposite?

Default: ~/.fluxbox/.fbrun_history
With patch: ~/.fbrun_history.

Didn't you switch files while making this patch? According to what I see ~/.fbrun_history becomes ~/.fluxbox/fbrun_history ... which is Allan pointing at.

--- main.cc    2008-09-19 20:56:40.000000000 -0300
+++ fluxbox-1.1.1/util/fbrun/main.cc    2008-01-15 07:50:53.000000000 -0200
[snip]
-        "   -hf [history file]          History file to load (default ~/.fbrun_history)"<<endl<<
+        "   -hf [history file]          History file to load (default ~/.fluxbox/fbrun_history)"<<endl<<
         "   -help                       Show this help"<<endl<<endl<<
         "Example: fbrun -fg black -bg white -text xterm -title \"run xterm\""<<endl;
 }
@@ -81,7 +81,7 @@
     string foreground("black");   // text color
     string background("white");   // text background color
     string display_name; // name of the display connection
-    string history_file("~/.fbrun_history"); // command history file
+    string history_file("~/.fluxbox/fbrun_history"); // command history file
kramerxiita wrote:

But is bettler make this patch optional too. I make this later.
But if anyone use Fluxbox, don't need install fbrun. So, why have .fluxbox directory?

Optional is good, although in this case it isn't illogical to drop the .fluxbox directory, as I said.

Patch make ~/.fbrun_history

Fluxbox default is ~/.fluxbox/fbrun_history

If don't use Fluxbox, only fbrun, i don't want fluxbox directory.

Patch is correct and make .fbrun_history without fluxbox directory.

So ~/.fluxbox/fbrun_history becomes  ~/.fbrun_history.

Install and you see.

Offline

#12 2008-09-25 11:55:55

foutrelis
Developer
From: Athens, Greece
Registered: 2008-07-28
Posts: 705
Website

Re: fbrun in AUR

The patch file is indeed going the opposite way. However, it looks correct because it applies reversed. Remove the --batch switch and you'll get a nice warning:

Reversed (or previously applied) patch detected!  Assume -R? [n]

Here's what the --batch switch does:

-t  or  --batch
          Suppress  questions  like  -f,  but make some different assumptions:
          skip patches whose headers do not contain file names  (the  same  as
          -f);  skip  patches for which the file has the wrong version for the
          Prereq: line in the patch; and assume that patches are  reversed  if
          they look like they are
.

Also, you don't have to accidentally the whole fluxbox just to get fbrun. Just flip lines 33-34 in the PKGBUILD so that it reads:

  cd ${srcdir}/fluxbox-1.1.1/util/fbrun
  make || return 1

As for the patch, do what you did the first time, but put "fluxbox-1.1.1/util/fbrun/main.cc" first and your modified "main.cc" second.

Last edited by foutrelis (2008-09-25 11:57:54)

Offline

#13 2008-09-25 14:14:40

kramerxiita
Member
Registered: 2008-02-14
Posts: 89

Re: fbrun in AUR

foutrelis wrote:

The patch file is indeed going the opposite way. However, it looks correct because it applies reversed. Remove the --batch switch and you'll get a nice warning:

Reversed (or previously applied) patch detected!  Assume -R? [n]

Here's what the --batch switch does:

-t  or  --batch
          Suppress  questions  like  -f,  but make some different assumptions:
          skip patches whose headers do not contain file names  (the  same  as
          -f);  skip  patches for which the file has the wrong version for the
          Prereq: line in the patch; and assume that patches are  reversed  if
          they look like they are
.

Also, you don't have to accidentally the whole fluxbox just to get fbrun. Just flip lines 33-34 in the PKGBUILD so that it reads:

  cd ${srcdir}/fluxbox-1.1.1/util/fbrun
  make || return 1

As for the patch, do what you did the first time, but put "fluxbox-1.1.1/util/fbrun/main.cc" first and your modified "main.cc" second.

Thanks, foutrelis.

Update with your tips.

Thanks, again.

Offline

Board footer

Powered by FluxBB