This is an archive of the discontinued Mercurial Phabricator instance.

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	31074		Nov 15 2021, 12:42 PM	★	★
Diff 2	31139		Nov 26 2021, 9:59 AM	★	★
Diff 3	31146	rHGa2e278b5e2655d551e02b66175a651ab81178667	Nov 9 2021, 1:09 PM	★	★
Diff 4	31159		Nov 26 2021, 10:32 AM	★	★

Commit	Parents	Author	Summary	Date
c8f7b3e1b860	5e887be991a1	Simon Sapin		Nov 9 2021, 1:09 PM

Status	Author	Revision
Closed	SimonSapin	D11815 rhg: Add support for `rhg status -n`
Closed	SimonSapin	D11774 rhg: Fix status desambiguation of symlinks and executable files
Closed	SimonSapin	D11773 rhg: Rename cat_file_is_modified
Closed	SimonSapin	D11772 rhg: Also parse flags in the manifest parser
Closed	SimonSapin	D11771 rhg: Propogate manifest parse errors instead of panicking
Closed	SimonSapin	D11769 tests: Adapt test-basic.t expected output for rhg
Closed	SimonSapin	D11762 rhg: Colored output is not supported
Closed	SimonSapin	D11761 rhg: [encode] and [decode] config sections are not supported
Closed	SimonSapin	D11760 rhg: Config commands.status.terse is not supported
Closed	SimonSapin	D11759 rhg: Propagate config errors in `rhg status`
Closed	SimonSapin	D11758 rhg: $HG_PENDING is not supported
Closed	SimonSapin	D11756 rhg: Enable `rhg status` in most tests

Diff 31139

	/// Return all keys defined for the given section			/// Return all keys defined for the given section
	pub fn get_section_keys(&self, section: &[u8]) -> HashSet<&[u8]> {			pub fn get_section_keys(&self, section: &[u8]) -> HashSet<&[u8]> {
	self.layers			self.layers
	.iter()			.iter()
	.flat_map(\|layer\| layer.iter_keys(section))			.flat_map(\|layer\| layer.iter_keys(section))
	.collect()			.collect()
	}			}

				/// Returns whether any key is defined in the given section
				pub fn has_non_empty_section(&self, section: &[u8]) -> bool {
				self.layers
				.iter()
				.any(\|layer\| layer.has_non_empty_section(section))
				}

	/// Get raw values bytes from all layers (even untrusted ones) in order			/// Get raw values bytes from all layers (even untrusted ones) in order
	/// of precedence.			/// of precedence.
	#[cfg(test)]			#[cfg(test)]
	fn get_all(&self, section: &[u8], item: &[u8]) -> Vec<&[u8]> {			fn get_all(&self, section: &[u8], item: &[u8]) -> Vec<&[u8]> {
	let mut res = vec![];			let mut res = vec![];
	for layer in self.layers.iter().rev() {			for layer in self.layers.iter().rev() {
	if let Some(v) = layer.get(&section, &item) {			if let Some(v) = layer.get(&section, &item) {
	res.push(v.bytes.as_ref());			res.push(v.bytes.as_ref());

	/// Returns the keys defined in the given section			/// Returns the keys defined in the given section
	pub fn iter_keys(&self, section: &[u8]) -> impl Iterator<Item = &[u8]> {			pub fn iter_keys(&self, section: &[u8]) -> impl Iterator<Item = &[u8]> {
	self.sections			self.sections
	.get(section)			.get(section)
	.into_iter()			.into_iter()
	.flat_map(\|section\| section.keys().map(\|vec\| &**vec))			.flat_map(\|section\| section.keys().map(\|vec\| &**vec))
	}			}

				/// Returns whether any key is defined in the given section
				pub fn has_non_empty_section(&self, section: &[u8]) -> bool {
				self.sections
				.get(section)
				.map_or(false, \|section\| !section.is_empty())
				}

	pub fn is_empty(&self) -> bool {			pub fn is_empty(&self) -> bool {
	self.sections.is_empty()			self.sections.is_empty()
	}			}

	/// Returns a `Vec` of layers in order of precedence (so, in read order),			/// Returns a `Vec` of layers in order of precedence (so, in read order),
	/// recursively parsing the `%include` directives if any.			/// recursively parsing the `%include` directives if any.
	pub fn parse(src: &Path, data: &[u8]) -> Result<Vec<Self>, ConfigError> {			pub fn parse(src: &Path, data: &[u8]) -> Result<Vec<Self>, ConfigError> {
	let mut layers = vec![];			let mut layers = vec![];

	check_extensions(config)?;			check_extensions(config)?;

	if std::env::var_os("HG_PENDING").is_some() {			if std::env::var_os("HG_PENDING").is_some() {
	// TODO: only if the value is `== repo.working_directory`?			// TODO: only if the value is `== repo.working_directory`?
	// What about relative v.s. absolute paths?			// What about relative v.s. absolute paths?
	Err(CommandError::unsupported("$HG_PENDING"))?			Err(CommandError::unsupported("$HG_PENDING"))?
	}			}

				if config.has_non_empty_section(b"encode") {
				Err(CommandError::unsupported("[encode] config"))?
				}

				if config.has_non_empty_section(b"decode") {
				Err(CommandError::unsupported("[decode] config"))?
				}

	Ok(())			Ok(())
	}			}

	TODO: fix rhg bugs that make this test fail when status is enabled
	$ unset RHG_STATUS


	Test encode/decode filters			Test encode/decode filters

	$ hg init			$ hg init
	$ cat > .hg/hgrc <<EOF			$ cat > .hg/hgrc <<EOF
	> [encode]			> [encode]
	> not.gz = tr [:lower:] [:upper:]			> not.gz = tr [:lower:] [:upper:]
	> *.gz = gzip -d			> *.gz = gzip -d
	> [decode]			> [decode]