mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] udev: 60-persistent-storage.rules missing
@ 2015-09-28 13:35 Matthias Klein
  2015-09-28 14:07 ` Juergen Borleis
  0 siblings, 1 reply; 4+ messages in thread
From: Matthias Klein @ 2015-09-28 13:35 UTC (permalink / raw)
  To: ptxdist


[-- Attachment #1.1: Type: text/plain, Size: 554 bytes --]

Hello,

I tried to update our BSP to ptxdist-2015.09.0 and now we are missing 
the "/dev/disk/by-path/" directory.
I found that the file "60-persistent-storage.rules" is missing. (We are 
using legacy udev and not systemd.)


The file 60-persistent-storage.rules is in the 
platform-xxx/packages/udev-182/lib/udev/rules.d/directory.
But not in the final udev_182_armhf.ipk package or the root directory.

With ptxdist-2015.07.0 it was working fine.


Does anyone has the same problem, or a tip what is wrong?


Best regards,
Matthias

[-- Attachment #1.2: Type: text/html, Size: 1813 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] udev: 60-persistent-storage.rules missing
  2015-09-28 13:35 [ptxdist] udev: 60-persistent-storage.rules missing Matthias Klein
@ 2015-09-28 14:07 ` Juergen Borleis
  2015-09-28 14:40   ` Matthias Klein
  0 siblings, 1 reply; 4+ messages in thread
From: Juergen Borleis @ 2015-09-28 14:07 UTC (permalink / raw)
  To: ptxdist, Matthias Klein

Hi Matthias,

On Monday 28 September 2015 15:35:59 Matthias Klein wrote:
> I tried to update our BSP to ptxdist-2015.09.0 and now we are missing
> the "/dev/disk/by-path/" directory.
> I found that the file "60-persistent-storage.rules" is missing. (We are
> using legacy udev and not systemd.)
>
> The file 60-persistent-storage.rules is in the
> platform-xxx/packages/udev-182/lib/udev/rules.d/directory.
> But not in the final udev_182_armhf.ipk package or the root directory.
>
> With ptxdist-2015.07.0 it was working fine.
>
> Does anyone has the same problem, or a tip what is wrong?

Hmm, seems a bug in "rules/udev.make". Please copy the file from your PTXdist 
installation into your local "rules/" directory and change the following:

diff --git a/rules/udev.make b/rules/udev.make
index 6e0e6ed3a092..bda096a012dd 100644
--- a/rules/udev.make
+++ b/rules/udev.make
@@ -98,7 +98,7 @@ UDEV_RULES-$(PTXCONF_UDEV_LEGACY) += \
 	42-usb-hid-pm.rules
 
 ifndef PTXCONF_SYSTEMD
-UDEV_RULES-y := \
+UDEV_RULES-y += \
 	60-persistent-serial.rules \
 	75-tty-description.rules \
 	95-udev-late.rules

Regards,
Juergen

-- 
Pengutronix e.K.                              | Juergen Borleis             |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] udev: 60-persistent-storage.rules missing
  2015-09-28 14:07 ` Juergen Borleis
@ 2015-09-28 14:40   ` Matthias Klein
  2015-09-28 14:46     ` Juergen Borleis
  0 siblings, 1 reply; 4+ messages in thread
From: Matthias Klein @ 2015-09-28 14:40 UTC (permalink / raw)
  To: Juergen Borleis, ptxdist

Hello Juergen,

thanks a lot for the tip!
I compared the files (by hand/eyes), but overlooked the "plus"...


Should I send a patch?


Best regards,
Matthias


------ Originalnachricht ------
Von: "Juergen Borleis" <jbe@pengutronix.de>
An: ptxdist@pengutronix.de; "Matthias Klein" 
<matthias.klein@optimeas.de>
Gesendet: 28.09.2015 16:07:36
Betreff: Re: [ptxdist] udev: 60-persistent-storage.rules missing

>Hi Matthias,
>
>On Monday 28 September 2015 15:35:59 Matthias Klein wrote:
>>  I tried to update our BSP to ptxdist-2015.09.0 and now we are missing
>>  the "/dev/disk/by-path/" directory.
>>  I found that the file "60-persistent-storage.rules" is missing. (We 
>>are
>>  using legacy udev and not systemd.)
>>
>>  The file 60-persistent-storage.rules is in the
>>  platform-xxx/packages/udev-182/lib/udev/rules.d/directory.
>>  But not in the final udev_182_armhf.ipk package or the root 
>>directory.
>>
>>  With ptxdist-2015.07.0 it was working fine.
>>
>>  Does anyone has the same problem, or a tip what is wrong?
>
>Hmm, seems a bug in "rules/udev.make". Please copy the file from your 
>PTXdist
>installation into your local "rules/" directory and change the 
>following:
>
>diff --git a/rules/udev.make b/rules/udev.make
>index 6e0e6ed3a092..bda096a012dd 100644
>--- a/rules/udev.make
>+++ b/rules/udev.make
>@@ -98,7 +98,7 @@ UDEV_RULES-$(PTXCONF_UDEV_LEGACY) += \
>   42-usb-hid-pm.rules
>
>  ifndef PTXCONF_SYSTEMD
>-UDEV_RULES-y := \
>+UDEV_RULES-y += \
>   60-persistent-serial.rules \
>   75-tty-description.rules \
>   95-udev-late.rules
>
>Regards,
>Juergen
>
>--
>Pengutronix e.K.                              | Juergen Borleis         
>     |
>Industrial Linux Solutions                    | 
>http://www.pengutronix.de/  |


-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] udev: 60-persistent-storage.rules missing
  2015-09-28 14:40   ` Matthias Klein
@ 2015-09-28 14:46     ` Juergen Borleis
  0 siblings, 0 replies; 4+ messages in thread
From: Juergen Borleis @ 2015-09-28 14:46 UTC (permalink / raw)
  To: Matthias Klein; +Cc: ptxdist

Hi Matthias,

On Monday 28 September 2015 16:40:41 Matthias Klein wrote:
> thanks a lot for the tip!
> I compared the files (by hand/eyes), but overlooked the "plus"...

:)

> Should I send a patch?

The patch is already in ptxdist-master since a few minutes...

Regards,
Juergen

-- 
Pengutronix e.K.                              | Juergen Borleis             |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |
-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-09-28 14:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-28 13:35 [ptxdist] udev: 60-persistent-storage.rules missing Matthias Klein
2015-09-28 14:07 ` Juergen Borleis
2015-09-28 14:40   ` Matthias Klein
2015-09-28 14:46     ` Juergen Borleis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox