Re: best way to add patch to x11/slim-1.3.1

2008-11-19 Thread Benjamin Lee
On 11/19/08 17:34, Fbsd1 wrote:
> Greg Larkin wrote:
>> -BEGIN PGP SIGNED MESSAGE-
>> Hash: SHA1
>>
>> Fbsd1 wrote:
>>> On the developers website there is a patch i want to apply
>>>
>>> http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663
>>>
>>>
>>>
>>> [ Patch #2283 ] Add a variable to run shutdown commands without root
>>> pass.
>>>
>>> How can i get "make install" to apply this patch while compiling the
>>> port?
>>>
>>
>> Hi Fbsd1,
>>
>> Since you've already found a unified diff of the change that you want to
>> incorporate into the port, you can submit a PR (problem report) using
>> the form here: http://www.freebsd.org/send-pr.html.
>>
>> Just follow the instructions on that page, and your patch will be
>> submitted to the PR system.  The port maintainer reviews your PR, makes
>> the necessary change and possibly updates the port's revision number.
>> You then use portupgrade or some other means to install the new version
>> of the port with the incorporated patch.
>>
>> Hope that helps,
>> Greg
>> - --
>> Greg Larkin
>>
> 
> I submitted PR like you suggested, But i am in need of more immediate
> results. What changes to the port files do i need to make to get the
> port to complie in the patch file?

You should read the FreeBSD Porter's Handbook [1].  In particular,
you'll probably be interested in section 4.4, "Patching" [2].

[1]
http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/index.html
[2]
http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/slow-patch.html


-- 
Benjamin Lee
http://www.b1c1l1.com/



signature.asc
Description: OpenPGP digital signature


Re: best way to add patch to x11/slim-1.3.1

2008-11-19 Thread matt donovan
On Wed, Nov 19, 2008 at 8:34 PM, Fbsd1 <[EMAIL PROTECTED]> wrote:

> Greg Larkin wrote:
>
>> -BEGIN PGP SIGNED MESSAGE-
>> Hash: SHA1
>>
>> Fbsd1 wrote:
>>
>>> On the developers website there is a patch i want to apply
>>>
>>>
>>> http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663
>>>
>>>
>>> [ Patch #2283 ] Add a variable to run shutdown commands without root
>>> pass.
>>>
>>> How can i get "make install" to apply this patch while compiling the
>>> port?
>>>
>>>
>> Hi Fbsd1,
>>
>> Since you've already found a unified diff of the change that you want to
>> incorporate into the port, you can submit a PR (problem report) using
>> the form here: http://www.freebsd.org/send-pr.html.
>>
>> Just follow the instructions on that page, and your patch will be
>> submitted to the PR system.  The port maintainer reviews your PR, makes
>> the necessary change and possibly updates the port's revision number.
>> You then use portupgrade or some other means to install the new version
>> of the port with the incorporated patch.
>>
>> Hope that helps,
>> Greg
>> - --
>> Greg Larkin
>>
>>
> I submitted PR like you suggested, But i am in need of more immediate
> results. What changes to the port files do i need to make to get the port to
> complie in the patch file?
>
>
> ___
> freebsd-questions@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-questions
> To unsubscribe, send any mail to "
> [EMAIL PROTECTED]"
>

If I remember correctly you just add the patch file to the files/ directory
under the port with a name like patch-
___
freebsd-questions@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-questions
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


Re: best way to add patch to x11/slim-1.3.1

2008-11-19 Thread Fbsd1

Greg Larkin wrote:

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Fbsd1 wrote:

On the developers website there is a patch i want to apply

http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663


[ Patch #2283 ] Add a variable to run shutdown commands without root pass.

How can i get "make install" to apply this patch while compiling the port?



Hi Fbsd1,

Since you've already found a unified diff of the change that you want to
incorporate into the port, you can submit a PR (problem report) using
the form here: http://www.freebsd.org/send-pr.html.

Just follow the instructions on that page, and your patch will be
submitted to the PR system.  The port maintainer reviews your PR, makes
the necessary change and possibly updates the port's revision number.
You then use portupgrade or some other means to install the new version
of the port with the incorporated patch.

Hope that helps,
Greg
- --
Greg Larkin



I submitted PR like you suggested, But i am in need of more immediate 
results. What changes to the port files do i need to make to get the 
port to complie in the patch file?


___
freebsd-questions@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-questions
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


Re: best way to add patch to x11/slim-1.3.1

2008-11-19 Thread Greg Larkin
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Fbsd1 wrote:
> On the developers website there is a patch i want to apply
> 
> http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663
> 
> 
> [ Patch #2283 ] Add a variable to run shutdown commands without root pass.
> 
> How can i get "make install" to apply this patch while compiling the port?
> 

Hi Fbsd1,

Since you've already found a unified diff of the change that you want to
incorporate into the port, you can submit a PR (problem report) using
the form here: http://www.freebsd.org/send-pr.html.

Just follow the instructions on that page, and your patch will be
submitted to the PR system.  The port maintainer reviews your PR, makes
the necessary change and possibly updates the port's revision number.
You then use portupgrade or some other means to install the new version
of the port with the incorporated patch.

Hope that helps,
Greg
- --
Greg Larkin

http://www.FreeBSD.org/   - The Power To Serve
http://www.sourcehosting.net/ - Ready. Set. Code.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.7 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJJBio0sRouByUApARAuaqAJ0dvk/SzKvcz/VzaFgSDuivb6RV3QCbBJqj
2iVzN4XzW92LpY6M34a5szM=
=C5Gj
-END PGP SIGNATURE-

___
freebsd-questions@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-questions
To unsubscribe, send any mail to "[EMAIL PROTECTED]"


best way to add patch to x11/slim-1.3.1

2008-11-19 Thread Fbsd1

On the developers website there is a patch i want to apply

http://developer.berlios.de/patch/?func=detailpatch&patch_id=2283&group_id=2663

[ Patch #2283 ] Add a variable to run shutdown commands without root pass.

How can i get "make install" to apply this patch while compiling the port?

This is the contents of the patch file

 From: Nicolas Pierron <[EMAIL PROTECTED]>
 Subject: r???: Add a variable to run shutdown commands without root 
password.


   URL: http://svn.berlios.de/svnroot/repos/slim/trunk

 ChangeLog:
 2007-12-16  Nicolas Pierron  <[EMAIL PROTECTED]>

Add a variable to run system command without root password.
* app.cpp: Add the test for reboot, halt and suspend.
* cfg.cpp: Add the new variable with the default value set to false.
* slim.conf: Add an example of the command.

 ---
  app.cpp   |5 +
  cfg.cpp   |1 +
  slim.conf |5 +
  3 files changed, 11 insertions(+)

 Index: slim.conf
 ===
 --- slim.conf  (revision 150)
 +++ slim.conf  (working copy)
 @@ -10,6 +10,11 @@
  console_cmd /usr/X11R6/bin/xterm -C -fg white -bg black +sb 
-T "Console login" -e /bin/sh -c "/bin/cat /etc/issue; exec /bin/login"

  #suspend_cmd/usr/sbin/suspend

 +# Let normal users have access to systems commands. If the value is true,
 +# then the root password is requiered to start a system command.
 +# Valid values: true|false
 +# root_password false
 +
  # Full path to the xauth binary
  xauth_path /usr/X11R6/bin/xauth

 Index: cfg.cpp
 ===
 --- cfg.cpp(revision 150)
 +++ cfg.cpp(working copy)
 @@ -36,6 +36,7 @@
  options.insert(option("login_cmd","exec /bin/bash -login 
~/.xinitrc %session"));

  options.insert(option("halt_cmd","/sbin/shutdown -h now"));
  options.insert(option("reboot_cmd","/sbin/shutdown -r now"));
 +options.insert(option("root_password","true"));
  options.insert(option("suspend_cmd",""));
  options.insert(option("sessionstart_cmd",""));
  options.insert(option("sessionstop_cmd",""));
 Index: app.cpp
 ===
 --- app.cpp(revision 150)
 +++ app.cpp(working copy)
 @@ -407,6 +407,11 @@
  case Panel::Console:
  cerr << APPNAME << ": Got a special command (" << 
LoginPanel->GetName() << ")" << endl;

  return true; // <--- This is simply fake!
 +case Panel::Suspend:
 +case Panel::Halt:
 +case Panel::Reboot:
 +if (cfg->getOption("root_password") == "false")
 +  return true;
  default:
  break;
  };


___
freebsd-questions@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-questions
To unsubscribe, send any mail to "[EMAIL PROTECTED]"