public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download: 
* Re: [gentoo-portage-dev] [PATCH v5] Test for read-only filesystems, bail out during preinst if there are any which will be written to and display a useful error message. Fixes bug 378869.
  @ 2014-01-22  5:24 99%   ` Mike Frysinger
  0 siblings, 0 replies; 1+ results
From: Mike Frysinger @ 2014-01-22  5:24 UTC (permalink / raw
  To: gentoo-portage-dev; +Cc: Chris Reffett

[-- Attachment #1: Type: Text/Plain, Size: 421 bytes --]

On Monday 20 January 2014 23:00:31 Chris Reffett wrote:
> +	if not ro_filesystems:
> +		return ro_filesystems
> +
> +	for directory in dir_list:
> +		for filesystem in ro_filesystems:
> +			if filesystem == directory:
> +				ro_filesystems_written.add(filesystem)
> +
> +	return ro_filesystems_written

is this just the intersection of two sets ?  so all this logic could be:
	return ro_filesystems ^ set(dir_list)
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[relevance 99%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2014-01-21  3:32     [gentoo-portage-dev] [PATCH v4] Test for read-only filesystems, bail out during preinst if there are any which will be written to and display a useful error message. Fixes bug 378869 Mike Frysinger
2014-01-21  4:00     ` [gentoo-portage-dev] [PATCH v5] " Chris Reffett
2014-01-22  5:24 99%   ` Mike Frysinger

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