[Xrdp-devel] Patch to fix SBINDIR issue

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Xrdp-devel] Patch to fix SBINDIR issue

Alan Messias
Hi developers,

Please, consider to apply this patch http://sprunge.us/SMdO

This is due to the fact that with the previous code we cannot port xrdp to every Linux distros. Eg.: I am trying to port it to Alpine Linux[1]


Thanks,
--
Atenciosamente,
Alan Messias Cordeiro

The nice thing about standards is that there are so many to choose from. And if you really don't like all the standards you just have to wait another year until the one arises you are looking for. -- A. Tanenbaum, "Introduction to Computer Networks"


------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
xrdp-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/xrdp-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Xrdp-devel] Patch to fix SBINDIR issue

Alan Messias
Hi Jay,

Infact we dont even need to remove pid files. I just kept this thinking that maybe there is a strong reason for this and tried to simplify the remove statement.
To be hosnet I really think that this should be dropped from the script - of course if there is no a strong reason for this.

Thanks,

On Thu, Jan 15, 2015 at 4:43 PM, Jay Sorg <[hidden email]> wrote:
Alpine looks like a good fit for xrdp.

I do see one error in the patch.
The last
+    find /var/run -name xrdp-sesman.pid -exec rm {} \;
is wrong I believe.

A summary might be
remove --noheading for ps
use "which" to locate the xrdp and xrd-sesman binaries.

But why do you change "if test" to "find" ?

Jay



On Thu, Jan 15, 2015 at 3:30 AM, Alan Messias <[hidden email]> wrote:
> Hi developers,
>
> Please, consider to apply this patch http://sprunge.us/SMdO
>
> This is due to the fact that with the previous code we cannot port xrdp to
> every Linux distros. Eg.: I am trying to port it to Alpine Linux[1]
>
>
> [1] http://www.alpinelinux.org
> Thanks,
> --
> Atenciosamente,
> Alan Messias Cordeiro
>
> The nice thing about standards is that there are so many to choose from. And
> if you really don't like all the standards you just have to wait another
> year until the one arises you are looking for. -- A. Tanenbaum,
> "Introduction to Computer Networks"
>
>
> ------------------------------------------------------------------------------
> New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
> GigeNET is offering a free month of service with a new server in Ashburn.
> Choose from 2 high performing configs, both with 100TB of bandwidth.
> Higher redundancy.Lower latency.Increased capacity.Completely compliant.
> http://p.sf.net/sfu/gigenet
> _______________________________________________
> xrdp-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/xrdp-devel
>



--
Atenciosamente,
Alan Messias Cordeiro

The nice thing about standards is that there are so many to choose from. And if you really don't like all the standards you just have to wait another year until the one arises you are looking for. -- A. Tanenbaum, "Introduction to Computer Networks"


------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
xrdp-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/xrdp-devel