Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-03-12 Thread Florian Festi
yup, wontfix.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-372293010___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-03-12 Thread Florian Festi
Closed #392.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#event-1516097945___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-17 Thread Jeff Johnson
Then you are stuck: it's your workload, not the rpmspec parser implementation, 
that is the root cause.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366440232___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-17 Thread Igor Gnatenko
If I would be querying them sequentially, then it would take hour or so

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366439874___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-17 Thread Igor Gnatenko
> Since your program appears to be multithreaded, try splitting the 4000 
> specfile queries across 10-20 worker threads.

That's exactly what it does.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366439857___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-17 Thread Igor Gnatenko
@n3npq it's not single spec.. Its 4000+ different specs.

What I wrote there if you run `rpmspec`, then tool is taking 2 minutes to do 
the job. If you remove execution of rpmspec, then tool takes 3 seconds.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366437766___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-17 Thread Igor Gnatenko
@n3npq believe me, it is slowest part in my tool ;)

```
287.25user 152.95system 2:11.94elapsed 333%CPU (0avgtext+0avgdata 
68724maxresident)k
13648inputs+0outputs (53major+30349595minor)pagefaults 0swaps
```

```diff
diff --git a/src/main.rs b/src/main.rs
index 4b0f2ff..9306517 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -79,6 +79,7 @@ fn run(opt: ) -> Result<(), Error> {
 let atime = author.when();
 let datetime =
 FixedOffset::east(atime.offset_minutes() * 3600 / 
60).timestamp(atime.seconds(), 0);
+/*
 let output = Command::new("rpmspec")
 .args(&[
 "--srpm",
@@ -92,6 +93,7 @@ fn run(opt: ) -> Result<(), Error> {
 workdir.join().to_str().unwrap(),
 ])
 .output()?;
+*/
 
 let mut chlog_header = format!(
 "* {} {} <{}>",
@@ -99,11 +101,13 @@ fn run(opt: ) -> Result<(), Error> {
 author.name().unwrap_or("Nobody"),
 author.email().unwrap_or("nob...@fedoraproject.org"),
 );
+/*
 if output.status.success() {
 chlog_header.push_str(!(" - {}", 
String::from_utf8_lossy()));
 } else {
 eprintln!("{}", String::from_utf8_lossy());
 }
+*/
 let chlog_entry = format!("- {}", commit.summary().unwrap());
 
 Ok(Some(format!("{}\n{}", chlog_header, chlog_entry)))
```

```
13.41user 2.05system 0:03.20elapsed 482%CPU (0avgtext+0avgdata 
66184maxresident)k
0inputs+0outputs (0major+495394minor)pagefaults 0swaps
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366431860___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] RFE: implement fast mode for getting EVR (#392)

2018-02-16 Thread Jeff Johnson
Um, spec file parsing is not exactly slow (even with lightly used %include) and 
there already is a means to query EVR from a spec file.

Presumably you are seeking the EVR used on every %changelog entry by brute 
force: try being more clever, by (say) pickling a store in the git directory 
incrementally.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/392#issuecomment-366388365___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint